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: Stig Bjørlykke <stig@xxxxxxxxxxxxx>
Date: Sun, 28 Jul 2013 21:36:50 +0200
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.


-- 
Stig Bjørlykke