<div dir="ltr">I personally like foo_, but m_ is fine.  I worry about unexpected side effects of changing member naming with private members.  I don't think there will be any if everything builds, but I still worry.  If we go for changing publics, we should pick a time and go for a major version number bump and expect a lot of peopIe using GDAL to go through a lot of pain.  I see a lot of stuff generally like this...<div><br></div><div>class Foo</div><div>{</div><div>public:<br></div><div>  Foo(int blahIn) { this->blah = blahIn; };</div><div>  int blah;</div><div>}</div><div><br></div><div>After looking at a lot of GDAL code, it seems that we should go with:</div><div><br></div><div>class Foo</div><div>{</div><div>  int m_blah;</div><div>public:</div><div> Foo( int blah ) : m_blah(blah) {}</div><div> int blah() { return m_blah; }</div><div>}</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 16, 2015 at 4:50 PM, Mateusz Loskot <span dir="ltr"><<a href="mailto:mateusz@loskot.net" target="_blank">mateusz@loskot.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 16 October 2015 at 22:47, Even Rouault <<a href="mailto:even.rouault@spatialys.com">even.rouault@spatialys.com</a>> wrote:<br>
><br>
>> I'm not proposing any changes to naming convention. AFAICT, currently GDAL<br>
>> either uses m_ prefix or nothing.<br>
><br>
> Well, I think you'll agree that using m_foo or bla_ is a naming convention,<br>
<br>
</span>Right, I meant, to actually apply unified convention: some classes use<br>
no prefix,<br>
others use m_. Let's choose m_ and stick to it.<br>
<span class=""><br>
> and there's nothing specified in<br>
> <a href="https://trac.osgeo.org/gdal/wiki/rfc8_devguide" rel="noreferrer" target="_blank">https://trac.osgeo.org/gdal/wiki/rfc8_devguide</a> related to that.<br>
<br>
</span>Then, it's gonna be updated, I think.<br>
<span class="im HOEnZb"><br>
Best regards,<br>
--<br>
Mateusz  Loskot, <a href="http://mateusz.loskot.net" rel="noreferrer" target="_blank">http://mateusz.loskot.net</a><br>
</span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org">gdal-dev@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/gdal-dev" rel="noreferrer" target="_blank">http://lists.osgeo.org/mailman/listinfo/gdal-dev</a></div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">--<div><a href="http://schwehr.org" target="_blank">http://schwehr.org</a></div></div>
</div>