On Mon, Aug 16, 2010 at 2:43 PM, Robert Story <[email protected]> wrote:

> On Mon, 16 Aug 2010 12:09:22 +0200 Bart wrote:
> BVA> Seems like a good idea to me. The only comment I have about the patch
> is
> BVA> that 'char' is a strange datatype choice for the boolean variable
> hex2digit.
> BVA> Why not to declare it as an integer ?
>
> Well, it's not a boolean, as it needs to hold 3 values (needs-init, true,
> false). And I just don't see the point of using 4 bytes when I really only
> need 3 bits.
>

Because that would allow you to write -1 instead of (char)-1 -- that's one
ugly cast less.

Bart.
------------------------------------------------------------------------------
This SF.net email is sponsored by 

Make an app they can't live without
Enter the BlackBerry Developer Challenge
http://p.sf.net/sfu/RIM-dev2dev 
_______________________________________________
Net-snmp-coders mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders

Reply via email to