Wireshark mailing list archives
Re: [Wireshark-commits] rev 48633: /trunk/epan/ /trunk/epan/: value_string.c
From: mmann78 () netscape net
Date: Thu, 28 Mar 2013 19:48:37 -0400 (EDT)
I've tried to clean up most of them (along with a few other of the decode_* functions) to be replaced with proto_tree_add_item (with masks in the hf items as needed), but I think the few remaining were a bit hard to decipher without a sample capture (which I didn't have). If I could get rid of all of them, I would have to "encourage" the use of proto_tree_add_item. -----Original Message----- From: Evan Huus <eapache () gmail com> To: Wireshark Developer List <wireshark-dev () wireshark org> Sent: Thu, Mar 28, 2013 6:15 pm Subject: Re: [Wireshark-dev] [Wireshark-commits] rev 48633: /trunk/epan/ /trunk/epan/: value_string.c On Thu, Mar 28, 2013 at 6:02 PM, <eapache () wireshark org> wrote:
http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=48633 User: eapache Date: 2013/03/28 03:02 PM Log: Greatly clean up value_string.c - use consistent indentation and coding style - add modelines - replace some g_assert calls with DISSECTOR_ASSERT where it makes sense - group related functions together and simplify many comments by referring
to
the 'normal' value_string as the canonical version Directory: /trunk/epan/ Changes Path Action +366 -328 value_string.c Modified
Tangential to this, does anybody know what the deal is with decode_enumerated_bitfield() and decode_enumerated_bitfield_shifted()? The first is called in exactly one place, the second not at all. They return static buffers (which is odd, though not necessarily wrong, perhaps they should be using packet-scope memory?), and while they make use of value strings they don't seem immediately value-string related. The only one that is called is fairly short so I'm tempted to manually inline that and drop both functions. At the very least they should probably be moved to to_str.c (or somewhere else). Thoughts? Evan ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Re: [Wireshark-commits] rev 48633: /trunk/epan/ /trunk/epan/: value_string.c Evan Huus (Mar 28)
- Re: [Wireshark-commits] rev 48633: /trunk/epan/ /trunk/epan/: value_string.c mmann78 (Mar 28)
- Re: [Wireshark-commits] rev 48633: /trunk/epan/ /trunk/epan/: value_string.c Guy Harris (Mar 28)
- Re: [Wireshark-commits] rev 48633: /trunk/epan/ /trunk/epan/: value_string.c Evan Huus (Mar 28)