Christian Thalinger <[EMAIL PROTECTED]> writes:

> The patch looks ok to me.  

Great; thanks for reviewing it.

> Maybe it should be an uint32_t, as this is the data type from the
> union.

No, it needs to be the corresponding signed type, so that the
subsequent comparison of L against 0 will yield correct results.

-- 
Aaron M. Ucko, KB1CJC (amu at alum.mit.edu, ucko at debian.org)
Finger [EMAIL PROTECTED] (NOT a valid e-mail address) for more info.

Reply via email to