Ah, sh**
On Sonntag, 14. Dezember 2014 17:12:23 CEST, Thomas Lübking wrote:
On Sonntag, 14. Dezember 2014 16:43:10 CEST, Nuno Hultberg wrote:
Thomas, I'll add markReplyModeHandpicked(). I am using kate.
Settings/Configure Editor/Editing/Indentation/->Spaces: "4 characters"
I suggest to also highlight tabs and trailing spaces in the
"Appearance" page.
Jan, It was not me who wrote the inconsistent code. I also do not
understand why it is inconsistent as the else if clauses look the same for
me.
I think Jan would like to see "markReplyModeHandpicked()"
returning bool (unconditionally true) and assign the flag there
as well.
One could also twist the code instead, starting w/ a false
flag, trying to setReplyMode() and if the flag is false at the
end of that, markReplyModeHandpicked()
Cheers,
Thomas
PS: I'd suggest to usually keep patch related discussions in gerrit
PPS: github allows to mail back ;-P