<div dir="ltr">Thanks Even for driving this to completion!<div><br></div><div>To follow up, the RFC's list of core changes are pretty limited:</div><div><br></div><div><div>- Change <a href="http://configure.ac">configure.ac</a> to remove with_cpp11 flag, always use C++11, and fail if AX_CXX_COMPILE_STDCXX_11 does not find C++11</div><div>- Remove @CXX11_SUPPORT@ in <a href="http://GDALmake.opt.in">GDALmake.opt.in</a> and anywhere else it occurs</div><div>- Remove continuous build targets that do not support C++11 from Travis-CI and AppVeyor?</div><div>- Remove #if HAVE_CXX11 and leave only the C++11 code</div><div>- find . -name \*.h -o -name \*.cpp | xargs egrep 'HAVE_CXX11'</div><div>- CPL_STATIC_ASSERT -> static_assert</div><div>- NULL -> nullptr (Only for C++ code)</div><div>- CPL_OVERRIDE -> override and remove redundant virtual</div><div>- MAX or -max() -> std::numeric_limits<T>::lowest()</div></div><div><br></div><div> It would be worthwhile to (even if only briefly) discuss the use each new feature before starting to committing changes.  That will help us add to a GDAL style guide over time.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 8, 2017 at 8:42 AM, Even Rouault <span dir="ltr"><<a href="mailto:even.rouault@spatialys.com" target="_blank">even.rouault@spatialys.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><u></u>
<div style="font-family:'Sans Serif';font-size:9pt;font-weight:400;font-style:normal">
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">Hi,</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">I've just committed in trunk the changes, initiated by Kurt, needed to enforce the C++11 build requirement</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">( <a href="https://trac.osgeo.org/gdal/wiki/rfc68_cplusplus11" target="_blank">https://trac.osgeo.org/gdal/<wbr>wiki/rfc68_cplusplus11</a> )</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">Please have a close look at <a href="https://travis-ci.org/OSGeo/gdal/builds" target="_blank">https://travis-ci.org/OSGeo/<wbr>gdal/builds</a> and <a href="https://ci.appveyor.com/project/OSGeo/gdal" target="_blank">https://ci.appveyor.com/<wbr>project/OSGeo/gdal</a> when using new C++11 features.</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">I think it would be good if we could restrict ourselves for now to things that VS 2013 can compile. See</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"><a href="https://msdn.microsoft.com/en-us/library/hh567368.aspx" target="_blank">https://msdn.microsoft.com/en-<wbr>us/library/hh567368.aspx</a></p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">Even</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"> </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">-- </p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px">Spatialys - Geospatial professional services</p>
<p style="margin-top:0px;margin-bottom:0px;margin-left:0px;margin-right:0px;text-indent:0px"><a href="http://www.spatialys.com" target="_blank">http://www.spatialys.com</a></p></div><br>______________________________<wbr>_________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org">gdal-dev@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/gdal-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/<wbr>mailman/listinfo/gdal-dev</a><br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">--<div><a href="http://schwehr.org" target="_blank">http://schwehr.org</a></div></div>
</div>