Wireshark-bugs: [Wireshark-bugs] [Bug 2689] [PATCH]: enhanced "what's past last mpls label?" heu
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=2689
--- Comment #5 from Jaap Keuter <jaap.keuter@xxxxxxxxx> 2008-07-13 10:36:43 PDT ---
I have committed the changes, since I didn't want to delay the expansion of
this
functionality, but i would strongly suggest the modernization of the code:
* Many items are retrieved from the TVB into a variable and then added with a
formating function, like _add_uint_format(). The same can be achieved by
properly filling out the header field definition and then adding the entry
using _add_item(). I haven't seen any with which this couldn't be done.
* The use of hidden fields, as mechanism to filter on, is deprecated for a
reason. Without studying the sourcecode, or guesswork, it's impossible to know
how to write a filter expression. If you think you need to resort to hidden
fields, your display design is wrong and needs to be reconsidered.
* Checking protocol violations while dissecting is a good thing, but reporting
it should be done via the expert info system. This has several advantages over
simply adding text items.
Please take these suggestions to hart, we would welcome any updates.
Committed revision 25730.
--
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
