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 /doc | |
| parent | c17893650c1691975e799e10148334f7e02ddc24 (diff) | |
Add CppCheck to the list of things to run before submitting code for review.
svn path=/trunk/; revision=49077
Diffstat (limited to 'doc')
| -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>. |