I see no need to add /* lp:885304 */ as a note in the source code. The function name makes it obvious what it's doing
Some style corrections: instead of
if (!main.activeFocus) main.focus = true
please use either
or
if (!main.activeFocus) { main.focus = true }
Also please consult the CODING document for the use of braces with if/else blocks in C++, this is wrong:
} else {
Would you have any comment on why Alt+F1 isn't working while you are in the Spread?
« Back to merge proposal
I see no need to add /* lp:885304 */ as a note in the source code. The function name makes it obvious what it's doing
Some style corrections: instead of
if (!main.activeFocus)
main.focus = true
please use either
if (!main.activeFocus) main.focus = true
or
if (!main.activeFocus) {
main.focus = true
}
Also please consult the CODING document for the use of braces with if/else blocks in C++, this is wrong:
}
else {
Would you have any comment on why Alt+F1 isn't working while you are in the Spread?