<div dir="ltr">My recommendation with clang-tidy is to start with all warnings disabled.  Then pick one that looks to be more likely to have constructive warnings.  Work on just those.  Then you can progressively add them.  And don't try to deal with warnings for anything involving swig.  </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 10, 2021 at 12:34 PM Andrew C Aitchison <<a href="mailto:andrew@aitchison.me.uk">andrew@aitchison.me.uk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
I've discovered clang-tidy and am letting it loose on my gdal drives.<br>
It is finding bugs and teaching me a lot about modern C++<br>
but as well as 100k+ bugs it found but did not report in "non-user code"<br>
it reports problems where I have used macros from the GDAL/CPL headers.<br>
<br>
I will report or submit patches once I understand these issues;<br>
in the mean time, are there any clang-tidy configs for gdal ?<br>
<br>
I have read and will reread <br>
<a href="https://erouault.blogspot.com/2016/01/software-quality-improvements-in-gdal.html" rel="noreferrer" target="_blank">https://erouault.blogspot.com/2016/01/software-quality-improvements-in-gdal.html</a><br>
<br>
Thanks,<br>
<br>
-- <br>
Andrew C. Aitchison                                     Kendal, UK<br>
                        <a href="mailto:andrew@aitchison.me.uk" target="_blank">andrew@aitchison.me.uk</a><br>
_______________________________________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org" target="_blank">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/mailman/listinfo/gdal-dev</a><br>
</blockquote></div>