widelands-dev team mailing list archive

[Hans Joachim Desserud <launchpad@xxxxxxxxxxxx>]

Hans Joachim Desserud has proposed merging lp:~hjd/widelands/cppcheck-silence-configurations into lp:widelands.

Requested reviews:
  Widelands Developers (widelands-dev)

For more details, see:
https://code.launchpad.net/~hjd/widelands/cppcheck-silence-configurations/+merge/254494

Strip out lines which mentions configurations which are the same as ones already covered. See comment #23 and #24 in bug 986611 for a rough comparison (two months apart, but the main change should be visible)
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~hjd/widelands/cppcheck-silence-configurations into lp:widelands.

=== modified file 'utils/create_cppcheck_report'
--- utils/create_cppcheck_report	2014-06-21 15:27:34 +0000
+++ utils/create_cppcheck_report	2015-03-29 13:57:30 +0000
@@ -8,7 +8,7 @@
 cppcheck --version                                                                     >> $FILE
 echo "</h2>"                                                                           >> $FILE
 echo "<div>"                                                                           >> $FILE
-cppcheck --force --quiet --verbose --enable=all -I src src 2>&1 | sed "s@^\[\(.*\):\([[:digit:]]\+\)\]: \(.*\)\$@<a href=\"http://bazaar.launchpad.net/%7Ewidelands-dev/widelands/trunk/annotate/head%3A/\1\?#L\2\";>\1:\2</a>: \3<br/>@" >> $FILE
+cppcheck --force --quiet --verbose --enable=all -I src src 2>&1 | grep -v "was not checked because its code equals another one" | sed "s@^\[\(.*\):\([[:digit:]]\+\)\]: \(.*\)\$@<a href=\"http://bazaar.launchpad.net/%7Ewidelands-dev/widelands/trunk/annotate/head%3A/\1\?#L\2\";>\1:\2</a>: \3<br/>@" >> $FILE
 echo "</div>"                                                                          >> $FILE
 echo "</body>"                                                                         >> $FILE
 echo "</html>"                                                                         >> $FILE