diff options
author | Evan Huus <eapache@gmail.com> | 2013-04-27 23:11:06 +0000 |
---|---|---|
committer | Evan Huus <eapache@gmail.com> | 2013-04-27 23:11:06 +0000 |
commit | b39cf04f14d4b9d16e1f4eca98c82f247fc186b6 (patch) | |
tree | ac8bf80190e5442b1f67ed3bcde50f89c6fbe2c7 | |
parent | c17893650c1691975e799e10148334f7e02ddc24 (diff) |
Add CppCheck to the list of things to run before submitting code for review.
svn path=/trunk/; revision=49077
-rw-r--r-- | doc/README.developer | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/doc/README.developer b/doc/README.developer index 125b7e26ab..10d4e6af6e 100644 --- a/doc/README.developer +++ b/doc/README.developer @@ -2775,6 +2775,11 @@ to the DISSECTOR_SRC section of epan/CMakeLists.txt problems: perl <wireshark_root>/tools/checkfiltername.pl <source-filename(s)> + - CHECK your dissector with CppCheck (http://cppcheck.sourceforge.net/) using + Wireshark's customized configuration. This is particularly important on + Windows, since Microsoft's compiler warnings are quite thin: + ./tools/cppcheck/cppcheck.sh <source-filename(s)> + - TEST YOUR DISSECTOR BEFORE SUBMITTING IT. Use fuzz-test.sh and/or randpkt against your dissector. These are described at <http://wiki.wireshark.org/FuzzTesting>. |