Re: [Wireshark-commits] rev 39559: /trunk/epan/ /trunk/epan/: proto.c

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

On Oct 25, 2011, at 11:43 AM, Jeff Morriss wrote:

> also require modifying hfinfo_numeric_format() to be able to generate representations of negative hexadecimal numbers

I'd say the representation should just be an unsigned representation.  I.e., for BASE_HEX and BASE_HEX_DEC, treat 
FT_INTn and FT_UINTn the same.  Change hfinfo_numeric_value_format() the same way.

> (so as to prevent "Apply as filter" on a signed BASE_HEX_DEC field whose decimal value is negative from causing an 
> abort).
>
> If that's done we may as well let in BASE_HEX and BASE_OCT too.
>
> This stuff came in rev 24643/bug 1539.  (See the later comments in the bug for at least one developer who wasn't 
> happy about these checks being put in.)

His complaint was about BASE_DEC_HEX and BASE_HEX_DEC; he doesn't want, for example, a 16-bit -1 to show in hex as 
-0x0001, so it sounds as if BASE_HEX and, presumably, BASE_OCT should show the number as unsigned in either case, so I 
don't see much point in signed values with BASE_HEX or BASE_OCT - if the number is only displayed in hex or octal, I'd 
argue it's unsigned.
___________________________________________________________________________
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