mailing list archives
Re: Converting to new proto tree too complex/errorprone
From: Jakub Zawadzki <darkjames-ws () darkjames pl>
Date: Sat, 21 Dec 2013 13:09:28 +0100
On Sat, Dec 21, 2013 at 11:27:27AM +0100, Joerg Mayer wrote:
converting a dissector to the new proto tree api is still unnecessarily
complex (and I managed to break all buildbots in the course).
I've attached a diff between the result of the conversion script output
and the currently committed source. Can you please take a look whether
at least some of this can be improved (if the script had been in perl
I'd probably would have tried to fix some of these things myself).
It might be possible to replace proto_peekremote / add NEW_PROTO_TREE_API define before config.h include.
Still if we're no longer forced to have header_field_info in register routine, I was thinking we could put hfi just
You can check sample in packet-netlink-sock_diag.c, which I base on packet-mpeg-descriptor.c
Also Peter suggested some modification to create HFI_REG_VAR macro.
btw. recently I have been doing some research if it's possible to do this kind of optimization using clang/ gcc plugins.
Right now I have no code.
Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org>
mailto:wireshark-dev-request () wireshark org?subject=unsubscribe