Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 22586: /trunk/epan/dissectors/ /trun

From: Stig Bjørlykke <stig.bjorlykke@xxxxxxxxx>
Date: Thu, 23 Aug 2007 00:40:46 +0200
Den 23. aug. 2007 kl. 00.34 skrev Guy Harris:

That's adding one more layer, with what amounts to a copy of the value
underneath it.  Other than providing the raw offset, what advantages
does it offer?  (There, I think, are other dissectors that have
bitfields that aren't in a subtree; if the word containing all the
bitfields isn't specified as an item of its own in the protocol, I'm
not sure it needs to be put into the protocol tree.)

I think it's much more easy to read the leading text and the value if the details of the bitfields does not start the line. Ofcourse my personal opinion, but mostly I do not care about the bits.


--
Stig Bjørlykke