https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=3170
Jeff Morriss <jeff.morriss.ws@xxxxxxxxx> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jeff.morriss.ws@xxxxxxxxx
Status|NEW |RESOLVED
Resolution| |FIXED
--- Comment #18 from Jeff Morriss <jeff.morriss.ws@xxxxxxxxx> 2009-09-13 18:44:01 PDT ---
I made some changes to the dissector and checked it in. Particular changes
(that I remember :-)) include:
- don't call check_col() (it's no longer necessary)
- replace duplicated blurbs with NULL
- put all the collectd subtrees under one top-level tree
- add a few ett_ variables to keep the different trees separate
- replace tvb_ensure_length_remaining() with an offset of 0 with
tvb_reported_length()
I'm still not sure about using proto_tree_add_protocol_format() all the time...
That's supposed to only be for the protocol's top-level entry. But I don't
see the harm for now.
Thanks for your contribution!
--
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.