[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