Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 50935: /trunk/epan/ /trunk/epan/diss
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Evan Huus <eapache@xxxxxxxxx>
Date: Sun, 28 Jul 2013 15:56:44 -0400
On Sun, Jul 28, 2013 at 3:36 PM, Stig Bjørlykke <stig@xxxxxxxxxxxxx> wrote:
> On Fri, Jul 26, 2013 at 11:51 PM,  <eapache@xxxxxxxxxxxxx> wrote:
>>  Add 64-bit value strings and the appropriate tooling (including yet another
>>  overloaded use of the DISPLAY field).
>
> While looking at adding 64-bit value strings support for Lua I found
> that the tree entry contains double name entry:
>
>   "Value: Something (Value: 42)"
>
> The correct entry should be (like it is for 32-bit):
>
>   "Value: Something (42)"
>
>
> The bug is in fill_label_number64(), where the format should be
> without the leading "%s: " and hfinfo->name.

Thanks, fixed in r50982.