Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 48633: /trunk/epan/ /trunk/epan/: va
On Thu, Mar 28, 2013 at 6:02 PM, <eapache@xxxxxxxxxxxxx> 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
