On Tue, Nov 07, 2006 at 05:47:25PM +0100, Lars Gullik Bjønnes wrote:
> I think you should also change setWide to wide...
>
> void wide(bool wide_inset) { wide_inset_ = wide_inset; }
>
> either both setX and getX, or just X and X (I like this last one
> best.)
X and X is pretty confusing as one can never be certain whether
'foo(x)' is a setter for x or a const function taking an additional
argument.
x-x getX-setX setX-x
aethetics 0 0 -
line noise + - 0
clarity - + +
sum: 0 0 0
So all depends on the relative weight you put on the categories.
I actually like the setX/x combo as most function calls are usually
getters, so this is low noise overhead.
Andre'
