[PATCH] Add dive list view to main window

Henrik Brautaset Aronsen subsurface at henrik.synth.no
Thu Apr 11 13:28:01 PDT 2013


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?

Anyway, we should drop the prefixes.

Henrik


More information about the subsurface mailing list