<div dir="ltr">Hi,<br><br><div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Sep 26, 2015 at 6:14 PM, Michaël Lefèvre <span dir="ltr"><<a href="mailto:lefevre00@gmail.com" target="_blank">lefevre00@gmail.com</a>></span> wrote:<br><div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div>1. there is a 'missingConfig' error, recommanding us to run the check with --checkconfig<br></div></div></div></div></blockquote><div>Im assuming you meant 'missingInclude' here, and not 'missingConfig' ? (I couldnt find a 'missingConfig'). If I remember correctly, there are cppcheck options to make it look in additional (sub-)directory's for header files. I actually tried that option, but couldnt get it to work properly, not even when I included the directory's that contained the right header files. So I abandoned that approach. Feel free to try out those options and improve the script.<br>;)<br><br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div></div>2. we could minimized the results by removing check on qa sub-directories, which are supposed to be the place for unit tests.<br></div></div></div></blockquote><div>Again, if I recall correctly, there's an '-i directory' option that lets you exclude certain directory's. So I guess it would be pretty easy to do something like this : 'cppcheck -i qadir1 -i qadir2 -i qadir3'.<br><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>3. if you use a suppression-list to silence some reports, may be this one could be in the git repository so we can use it localy.<br></div></div></blockquote><div>There is a switch that determines what kind of 'bugs' to report on. I think that it is currently set to '--enable=all', and guess you could specify only certain types of bugs to report on using that.<br><br>I dont actually remember options to specify specific pieces of code to exclude; but then again, the man page says this :<br><br>--suppress=<spec><br><br>    Suppress a specific warning. The format of <spec> is: [error id]:[filename]:[line]. The [filename] and [line] are optional. [error id] may be * to suppress all warnings (for a specified file or files). [filename] may contain the wildcard characters * or ?.<br><br>--suppressions-list=<file><br><br>    Suppress warnings listed in the file. Each suppression is in the format of <spec> above. <br><br><br>Which I guess could do what you want, but you would have to manually administer that suppression list which you may not want to do.<br></div><div><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div><div><div><div><div><div><div class="gmail_extra">By the way, thank you for bringing this idea to reality.<br></div></div></div></div></div></div></div></div></div></div></blockquote><div>No problem, I liked doing it, and was glad that I could contribute to the LibreOffice project even though I cant write C/C++.<br><br></div></div><br><br><br></div><div class="gmail_extra">Hope this helps,<br><br><br></div><div class="gmail_extra">- Maarten<br><br></div></div></div>