Re: [Wireshark-commits] rev 44860: /trunk/ /trunk/asn1/camel/: camel.cnf /trunk/asn1/charging_ase/: packet-charging_ase-template.c /trunk/asn1/cmp/: packet-cmp-template.c /trunk/asn1/credssp/: ...

[Jakub Zawadzki <darkjames-ws () darkjames pl>]

On Tue, Sep 18, 2012 at 04:10:09PM -0400, Bill Meier wrote:
> I haven't paid any attention to this change
> ("yet another ... passing data ...").

I tried to discuss this in [1].

> 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 
> before).

Idea was to replace all dissector_t variants into one, which definetely would break API.
Simpler version of patch I commited after almost week (night of 4/09 -> evening of 10/09).

It took me some time do to it, but I'm open to revert this changes and add new
*_dissector_t (instead of replacing), but do we really want 5 different dissector types:
old-style, new-style, data-style, heur, heur-data?

[1] http://www.wireshark.org/lists/wireshark-dev/201209/msg00018.html
___________________________________________________________________________
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