<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
<span style="font-size:12.0pt;mso-ascii-font-family:Calibri;mso-fareast-font-family:
"Times New Roman";mso-hansi-font-family:Calibri;mso-bidi-font-family:Calibri;
color:black">> Was it not you how came up with the idea to reduce the false positives with specifying
the includes?<o:p></o:p></span></p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
<span style="font-size:12.0pt;mso-ascii-font-family:Calibri;mso-fareast-font-family:
"Times New Roman";mso-hansi-font-family:Calibri;mso-bidi-font-family:Calibri;
color:black"><o:p> </o:p></span></p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
<span style="font-size:12.0pt;mso-ascii-font-family:Calibri;mso-fareast-font-family:
"Times New Roman";mso-hansi-font-family:Calibri;mso-bidi-font-family:Calibri;
color:black">No, it was not my idea. On #cppcheck, I was told by danmar, the primary developer
of cppcheck, that our script is using cppcheck incorrectly. Without being passed the same include locations as we pass the compiler, we should expect a large amount of garbage.
<o:p></o:p></span></p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
<span style="font-size:12.0pt;mso-ascii-font-family:Calibri;mso-fareast-font-family:
"Times New Roman";mso-hansi-font-family:Calibri;mso-bidi-font-family:Calibri;
color:black"><o:p> </o:p></span></p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
<span style="font-size:12.0pt;mso-ascii-font-family:Calibri;mso-fareast-font-family:
"Times New Roman";mso-hansi-font-family:Calibri;mso-bidi-font-family:Calibri;
color:black">In fact, according to the developer, we should not get any False Postives if we call
cppcheck correctly. He encouraged me to file bug reports for any FP that remain, once cppcheck is being run properly.
<span style="mso-spacerun:yes"> </span><o:p></o:p></span></p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
<span style="font-size:12.0pt;mso-ascii-font-family:Calibri;mso-fareast-font-family:
"Times New Roman";mso-hansi-font-family:Calibri;mso-bidi-font-family:Calibri;
color:black"><o:p> </o:p></span></p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
<span style="font-size:12.0pt;mso-ascii-font-family:Calibri;mso-fareast-font-family:
"Times New Roman";mso-hansi-font-family:Calibri;mso-bidi-font-family:Calibri;
color:black">> The main point that this change seems to simply reduce the scope of cppcheck. If
this is the purpose then we can just run cppcheck on an empty file and so we won't see any issue (all false positives will disappear).<o:p></o:p></span></p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
<span style="font-size:12.0pt;mso-ascii-font-family:Calibri;mso-fareast-font-family:
"Times New Roman";mso-hansi-font-family:Calibri;mso-bidi-font-family:Calibri;
color:black"><o:p> </o:p></span></p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
<span style="font-size:12.0pt;mso-ascii-font-family:Calibri;mso-fareast-font-family:
"Times New Roman";mso-hansi-font-family:Calibri;mso-bidi-font-family:Calibri;
color:black">Again, No my goal is to improve the Signal-to-noise. FPs can be dangerous as in tdf#96089
and make it much harder to spot real issues. <o:p></o:p></span></p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
<span style="font-size:12.0pt;mso-ascii-font-family:Calibri;mso-fareast-font-family:
"Times New Roman";mso-hansi-font-family:Calibri;mso-bidi-font-family:Calibri;
color:black"><o:p> </o:p></span></p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
<span style="font-size:12.0pt;mso-ascii-font-family:Calibri;mso-fareast-font-family:
"Times New Roman";mso-hansi-font-family:Calibri;mso-bidi-font-family:Calibri;
color:black">Currently, I am in the process of comparing old cppcheck fixes with and without the
'-Iinclude' option. So far, the three that I have checked would not be filtered out. In other words, had we been calling cppcheck the way I propose, these issues would have been much easier for developers to spot(4000 vs 500).</span></p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
<br>
</p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
-Luke<br>
<span style="font-size:12.0pt;mso-ascii-font-family:Calibri;mso-fareast-font-family:
"Times New Roman";mso-hansi-font-family:Calibri;mso-bidi-font-family:Calibri;
color:black"></span><o:p></o:p></p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
<o:p> </o:p></p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;margin-bottom:0in;margin-bottom:.0001pt;line-height:
normal">
<span style="font-size:12.0pt;mso-ascii-font-family:Calibri;mso-fareast-font-family:
"Times New Roman";mso-hansi-font-family:Calibri;mso-bidi-font-family:Calibri;
color:black"><o:p> </o:p></span></p>
<p style="margin: 0in 0in 8pt; line-height: 107%; font-size: 11pt; font-family: "Calibri", sans-serif;">
<o:p> </o:p></p>
<br>
</div>
</body>
</html>