aboutsummaryrefslogtreecommitdiffstats
path: root/doc
diff options
context:
space:
mode:
authorUlf Lamping <ulf.lamping@web.de>2007-03-31 15:32:06 +0000
committerUlf Lamping <ulf.lamping@web.de>2007-03-31 15:32:06 +0000
commitb05d384d71fb802bcd3b309b02962c3308486fc6 (patch)
treeedf22242d5bba56952660272eba6c3d10c748d4a /doc
parent4321baacdcddaf2ed910bf929697194b8cf991c9 (diff)
add a small section, that code should be warning free
svn path=/trunk/; revision=21291
Diffstat (limited to 'doc')
-rw-r--r--doc/README.developer11
1 files changed, 10 insertions, 1 deletions
diff --git a/doc/README.developer b/doc/README.developer
index bcd06b1b69..a9a9203c3b 100644
--- a/doc/README.developer
+++ b/doc/README.developer
@@ -579,7 +579,7 @@ Wireshark uses the underscore_convention rather than the InterCapConvention for
function names, so new code should probably use underscores rather than
intercaps for functions and variable names. This is especially important if you
are writing code that will be called from outside your code. We are just
-trying to keep things consistent for other users.
+trying to keep things consistent for other developers.
1.1.5 White space convention.
@@ -602,6 +602,15 @@ utility on an existing file. If you run across wildly varying
indentation styles within the same file, it might be helpful to send a
note to wireshark-dev for guidance.
+1.1.6 Compiler warnings
+
+You should write code that is free of compiler warnings. Such warnings will
+often indicate questionable code and sometimes even real bugs, so it's best
+to avoid warnings at all.
+
+The compiler flags in the Makefiles are set to "treat warnings as errors",
+so your code won't even compile when warnings occur.
+
1.2 Skeleton code.
Wireshark requires certain things when setting up a protocol dissector.