Re: adding units

[Guy Harris <guy () alum mit edu>]

On Apr 15, 2014, at 2:18 PM, "John Dill" <John.Dill () greenfieldeng com> wrote:

>   /* Pick the proper format string */
>   if (hfinfo->type == FT_FRAMENUM) {
>     /*
>      * Frame numbers are always displayed in decimal.
>      */
>     format = "%u"; [no reason to apply a units field to FT_FRAMENUM imo]

It's an ordinal number, not a cardinal number, so no units.

>   } else {
>     switch (hfinfo->display) {
>       case BASE_DEC:
>         format = "%u"; [would become "%u %s", should be a no-brainer]

Would become "%u %s" *if there's a units string*.  Otherwise you'd get an extra blank after the number.
___________________________________________________________________________
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