diff options
author | Laurent Deniel <laurent.deniel@free.fr> | 2000-08-11 13:46:34 +0000 |
---|---|---|
committer | Laurent Deniel <laurent.deniel@free.fr> | 2000-08-11 13:46:34 +0000 |
commit | 2de43f748bf0c8bcb9051dd26353e1a696fddfdf (patch) | |
tree | cdca5af9f08803978d9b64481738f280121dfb76 /doc | |
parent | 6a480953a956a2486e0b618bb02bed56faeac7bf (diff) |
Miscellaneous code cleaning
- add <stdarg.h> or <varargs.h> in snprintf.h
and remove those inclusions in the other #ifdef NEED_SNPRINTF_H codes
- remove the check of multiple inclusions in source (.c) code
(there is a bit loss of _cpp_ performance, but I prefer the gain of
code reading and maintenance; and nowadays, disk caches and VM are
correctly optimized ;-).
- protect all (well almost) header files against multiple inclusions
- add header (i.e. GPL license) in some include files
- reorganize a bit the way header files are included:
First:
#include <system_include_files>
#include <external_package_include_files (e.g. gtk, glib etc.)>
Then
#include "ethereal_include_files"
with the correct HAVE_XXX or NEED_XXX protections.
- add some HAVE_XXX checks before including some system header files
- add the same HAVE_XXX in wiretap as in ethereal
Please forgive me, if I break something (I've only compiled and regression
tested on Linux).
svn path=/trunk/; revision=2255
Diffstat (limited to 'doc')
-rw-r--r-- | doc/README.developer | 16 |
1 files changed, 6 insertions, 10 deletions
diff --git a/doc/README.developer b/doc/README.developer index 8e088e92af..4c4534a9ea 100644 --- a/doc/README.developer +++ b/doc/README.developer @@ -1,4 +1,4 @@ -$Id: README.developer,v 1.13 2000/05/28 17:04:23 oabad Exp $ +$Id: README.developer,v 1.14 2000/08/11 13:46:34 deniel Exp $ This file is a HOWTO for Ethereal developers. It describes how to start coding a protocol dissector and the use some of the important functions and variables @@ -57,7 +57,7 @@ code inside is needed only if you are using the "snprintf()" function. -The "$Id: README.developer,v 1.13 2000/05/28 17:04:23 oabad Exp $" in the comment will be updated by CVS when the file is +The "$Id: README.developer,v 1.14 2000/08/11 13:46:34 deniel Exp $" in the comment will be updated by CVS when the file is checked in; it will allow the RCS "ident" command to report which version of the file is currently checked out. @@ -66,7 +66,7 @@ version of the file is currently checked out. * Routines for PROTONAME dissection * Copyright 2000, YOUR_NAME <YOUR_EMAIL_ADDRESS> * - * $Id: README.developer,v 1.13 2000/05/28 17:04:23 oabad Exp $ + * $Id: README.developer,v 1.14 2000/08/11 13:46:34 deniel Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs <gerald@unicom.net> @@ -95,6 +95,7 @@ version of the file is currently checked out. #include <stdio.h> #include <stdlib.h> +#include <string.h> #ifdef HAVE_SYS_TYPES_H # include <sys/types.h> @@ -104,17 +105,12 @@ version of the file is currently checked out. # include <netinet/in.h> #endif +#include <glib.h> + #ifdef NEED_SNPRINTF_H -# ifdef HAVE_STDARG_H -# include <stdarg.h> -# else -# include <varargs.h> -# endif # include "snprintf.h" #endif -#include <string.h> -#include <glib.h> #include "packet.h" #include "packet-PROTOABBREV.h" |