Home page logo
/

wireshark logo Wireshark mailing list archives

Re: Coding style and example dissector
From: Joerg Mayer <jmayer () loplof de>
Date: Wed, 18 Dec 2013 13:52:41 +0100

On Tue, Dec 17, 2013 at 05:55:30PM -0800, Michael Lum wrote:
Could someone please write a coding style section for the new dissectors and perhaps
point to the best example dissector.

doc/README.developer last sections (5. White space convention) more or less
is what we have.

Currently, many of the dissectors I have submitted are having arbitrary white space/style
changes made.

I completely understand changes, for bugs, API changes, and warnings missed because of cross-platform
builds.

But I don't understand the need to change FROM a consistent style to some other style.

Maybe the consistent form was not apparent to the person make these changes.
Adding a mode-line seems like a good way to prevent this sort of "arbitrary"
changes.

If you are the de facto "maintainer" of these dissectors and you don't feel
comfortable with these changes then maybe open a bug and ask for these changes
to be reverted - you have to feel comfortable with your code.
In case you can live with (more or less) any coding style as seems
to be the case for you then it's "only" time someone else might have spent doing
other things but no harm was done and no revert is necessary.

Ciao
      Jörg

-- 
Joerg Mayer                                           <jmayer () loplof de>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe

  By Date           By Thread  

Current thread:
[ Nmap | Sec Tools | Mailing Lists | Site News | About/Contact | Advertising | Privacy ]
AlienVault