On: Lessons learned from rewriting code

[roel_v]

Maybe there once was a breakpoint at the return false, or an assert, and it was just left like this in case the breakpoint is needed again.

[indentit → roel_v]

Modern debuggers generally support conditional breakpoints, but they often incur a high performance penalty, so this reasoning makes sense to me.