On Apr 15, 2014, at 2:18 PM, "John Dill" <[email protected]> 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 <[email protected]>
Archives: http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
mailto:[email protected]?subject=unsubscribe
