Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 44860: /trunk/ /trunk/asn1/camel/: c
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Bill Meier <wmeier@xxxxxxxxxxx>
Date: Tue, 18 Sep 2012 16:10:09 -0400
On 9/18/2012 3:35 PM, Jeff Morriss wrote:

darkjames@xxxxxxxxxxxxx wrote:

http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=44860

User: darkjames
Date: 2012/09/10 02:40 PM

Log:
 Initial commit to support yet another method of passing data between
dissectors.

 Add new parameter 'data' to heur_dissector_t and new_dissector_t, for
now it's always NULL


Does anybody have an idea of how to handle this in custom dissectors
which need to be compiled in both 1.8 and trunk (without branching said
dissectors--something I really would like to avoid)?

I'm contemplating putting a macro in packet.h like
DISSECTORS_HAVE_DATA_PARAM and adding the appropriate #ifdef/#else logic
in my custom dissectors.  Any objections?  Any better ideas?
___________________________________________________________________________


I haven't paid any attention to this change
("yet another ... passing data ...").

I gather that it's not backwards compatible ?
Um, that would seem to mean that many private custom dissectors will need to change (with a macro or whatever) when 1.10 is released (if not bfore). 

:(


Bill