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


Reply via email to