From b39cf04f14d4b9d16e1f4eca98c82f247fc186b6 Mon Sep 17 00:00:00 2001
From: Evan Huus <eapache@gmail.com>
Date: Sat, 27 Apr 2013 23:11:06 +0000
Subject: Add CppCheck to the list of things to run before submitting code for
 review.

svn path=/trunk/; revision=49077
---
 doc/README.developer | 5 +++++
 1 file changed, 5 insertions(+)

(limited to 'doc')

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>.
-- 
cgit v1.2.3