Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/common/lib/libprop



On Mon, Oct 16, 2006 at 11:03:27AM -0700, Jason Thorpe wrote:
> 
> On Oct 16, 2006, at 9:14 AM, Tom Spindler wrote:
> 
> >There's also other warnings/build breakages generated on x86_64 in  
> >lib/libprop
> >with _PROP_CALLOC in userland, "fixed" by changing
> >
> >-#define        _PROP_CALLOC(s, t)      calloc(1, (s))
> >+#define        _PROP_CALLOC(s, t)      calloc((size_t) 1, (s))
> >
> >in prop_object_impl.h - but I'll let you fix it as you deem  
> >appropriate.
> 
> This is a completely bogus fix.  IMO, lint(1) is out of line warning  
> about a constant argument that falls within the valid range of the  
> data type of that argument.

Agreed - I personally have a strong dislike for casts used to change
the type of integer expressions.

        David

-- 
David Laight: david%l8s.co.uk@localhost



Home | Main Index | Thread Index | Old Index