Stefan Schimanski wrote:
Am 28.05.2008 um 22:41 schrieb rgheck:
This code in MathMacroTemplate.cpp has to be wrong:
case LFUN_IN_MATHMACROTEMPLATE:
flag.enabled();
break;
That READS the flag, pointlessly. It doesn't write it. Maybe what's
wanted is flag.enabled(true)?
Yes, that would make more sense.
Btw, calling the method enabled(bool) is a bad choice. Why not
setEnabled(bool)? Then those mistakes would be obvious.
I think that's actually more in line with our current style. I'll make
the change.
rh
