Inconsistency between optfor value assignment and struct documentation

[ThirstyWraith]

While the PGlobalOptimizer optfor member is set as

if (strcmp(token, "speed") == 0) {
    opt->optfor = 2;
    return 1;
}
if (strcmp(token, "space") == 0 || strcmp(token, "size") == 0) {
    opt->optfor = 1;
    return 1;
}

The struct's comment says
UInt8 optfor; /* 0x03: 0=default, 1=speed, 2=space */
Verify which one is correct for PPC.
MIPS uses different values altogether, with 0=speed, 1=space.
PR #1 assumes space=1, speed=2 during argument parsing to be intended behavior and sets MIPS optfor as space, unless speed was specified, which mimics IDE default behavior.
ctx->prefsOptimizer.optfor = (ctx->prefsOptimizer.optfor == 2) ? 0 : 1;