On Mon, Sep 09, 2013 at 05:45:09PM +0200, Joerg Mayer wrote:
> I just wanted to get familiar with the new hf_ stuff and tried the script on 
> the
> doc/packet-PROTOABBREV.c and I got at least one thing that doesn't look right:
> The code of the converted file contains "THIS_HF_INIT =", which should 
> probably
> be "HFI_PROTOABBREV ="?
> The #define gets added at the very beginning of the file instead of after the
> initial comment(s) section.
> Are there any other things that are incorrect / not good?

- it seems to lack something simialar to:
  "#define DAYTIME_HFI_INIT HFI_INIT(proto_daytime)"
- "static &hfi_register_info hf[] = {" doesn't look good either

 ciao
     Jörg
-- 
Joerg Mayer                                           <[email protected]>
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 <[email protected]>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:[email protected]?subject=unsubscribe

Reply via email to