<div dir="ltr">Hello Luke,<br><br>I'm not sure what your modification is doing. I just checked the report you attached and I compared it with the full report and I see your change filters out not only false positives. I used to use the cppcheck report to give my students a small task for their first patch. So it would be helpful to keep the full report (it can be kept next to the short one I guess), otherwise I would need to run cppcheck myself.<br><br>Thanks,<br>Tamás</div><br><div class="gmail_quote"><div dir="ltr">Luke Benes <<a href="mailto:lukebenes@hotmail.com">lukebenes@hotmail.com</a>> ezt írta (időpont: 2018. szept. 30., V, 5:20):<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Maarten,<br>
Thanks for your suggestion here and your earlier contributions to the Cppcheck Report. I agree that we should create the include file dynamically. However the approach used by your script seems like overkill. Cppcheck already finds that quoted includes like<br>
#include "GraphicExportFilter.hxx"<br>
.<br>
Also when there seems to have been a coding style that all <> includes outside of /inc folders should be defined by their relative path. Cppcheck only complains about 4 missing includes that do not follow this pattern.(see my earlier email on oddball includes).<br>
<br>
Unless, I'm missing something, I still prefer this approach:<br>
$ find . -type d \( -name "inc" -o -name "include" \) |sort > inc.txt<br>
<br>
inc.txt only has ~200 entries, where as  /tmp/tmpfile.txt has ~1,800 after sorting it.<br>
<br>
-Luke<br>
<br>
_______________________________________________<br>
LibreOffice mailing list<br>
<a href="mailto:LibreOffice@lists.freedesktop.org" target="_blank">LibreOffice@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/libreoffice" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
</blockquote></div>