Wireshark mailing list archives
Re: Remove hf_ variables
From: Graham Bloice <graham.bloice () trihedral com>
Date: Wed, 14 Aug 2013 09:26:17 +0100
On 13 August 2013 17:53, Jakub Zawadzki <darkjames-ws () darkjames pl> wrote:
On Fri, Aug 09, 2013 at 08:31:02PM +0200, Jakub Zawadzki wrote:On Wed, Aug 07, 2013 at 08:10:21PM +0200, Jakub Zawadzki wrote:I wrote some ugly gawk program for hf_ variables. Here is one + some patch for epan: http://www.wireshark.org/~darkjames/proto-new/0001-changes.txt Here is convertion of some dissectors (not quite automagic): http://www.wireshark.org/~darkjames/proto-new/0002-rewrite.txt We probably don't want this redefine macros inside proto-new.h butit's *much faster*to test.Removed in v2: http://www.wireshark.org/~darkjames/proto-new-v2/0001-changes.txt http://www.wireshark.org/~darkjames/proto-new-v2/0002-rewrite.txt Changes from v1: - proto_tree_add_item() now takes pointer to header_field_info *, if NEW_PROTO_TREE_API NOT defined small compatilibity macro is used. - convert names just to hfi_ not hfi_hf_ - fix proto_get_first_protocol_field(), proto_get_next_protocol_field()Ping, any feedback? Also missing final approval/reject (/me smiles to Gerald) ;]
It's not entirely clear to me from the thread what the current intent is. Your current plan is just the removal of hf vars? You'll also need to update the docs, README.dissector is the most obvious, then everyone's presentations on how to build a dissector :-)
___________________________________________________________________________ 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
Current thread:
- Re: Remove hf_ variables Bálint Réczey (Aug 06)
- Re: Remove hf_ variables Jakub Zawadzki (Aug 06)
- <Possible follow-ups>
- Re: Remove hf_ variables Guy Harris (Aug 06)
- Re: Remove hf_ variables Jeff Morriss (Aug 07)
- Re: Remove hf_ variables Jakub Zawadzki (Aug 07)
- Re: Remove hf_ variables Jakub Zawadzki (Aug 07)
- Re: Remove hf_ variables Jakub Zawadzki (Aug 09)
- Re: Remove hf_ variables Jakub Zawadzki (Aug 13)
- Re: Remove hf_ variables Graham Bloice (Aug 14)
- Re: Remove hf_ variables Jakub Zawadzki (Aug 14)
- Re: Remove hf_ variables Jakub Zawadzki (Aug 16)
- Re: Remove hf_ variables Jakub Zawadzki (Aug 09)