[PATCH] Add dive list view to main window
Dirk Hohndel
dirk at hohndel.org
Thu Apr 11 16:18:58 PDT 2013
Henrik Brautaset Aronsen <subsurface at henrik.synth.no> writes:
> Amit Chaudhuri wrote:
>> m prefix is not on local variables. I agree that would be odd. The
>> distinction being made with m (or m_ or _) is to mark private members
>> variables. It's quite common in C++ stuff I believe. But different
>> people make different calls. I'm not fussed either way. Maybe such a
>> mixed case we should drop it completely.
>
>
> I thought mVariable were member variables and _variable (or preferably
> variable_) were private variables?
DEFINITELY no leading '_' ...
> Anyway, we should drop the prefixes.
Yes.
/D
More information about the subsurface
mailing list