https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7726
--- Comment #13 from J. Bruce Fields <bfields@xxxxxxxxxxxx> 2012-09-18 04:53:03 PDT ---
(In reply to comment #12)
> I combined all of your patches into one, cleaning up along the way. Can you
> verify this new patch is what you're expecting?
Looks right. Also built and browsed through some of the calls and didn't see
any problems.
> I also noticed a bunch of proto_tree_add_text() calls within the dissector that
> could be changed to proto_tree_add_item calls so that the fields are
> filterable. Thanks for the current patches and feel free to continue to update
> the NFS dissector in that regard much like you did here (but as a separate bug
> report).
Sounds like a good idea, I'll try to make time for that some day.
Thanks for your help!
--
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.