Re: recent libalias changes

From: David Malone <dwmalone_at_maths.tcd.ie>
Date: Tue, 6 Jul 2004 12:48:18 +0100
On Tue, Jul 06, 2004 at 01:25:45PM +0200, des_at_des.no wrote:
> the code I replaced has casts on the lvalue, which is not allowed.

The cast seems to be in the expression leading to the lvalue, but
not on the lvalue itself. That is,

	((char *)p)[0] = 1;	/* cast in expression */
	(char)p[1] = 1;		/* cast in lvalue */

I think the former is permitted, though it may not always be wise.

> >                     The while loop that is immediately above it
> > looks risky though, as this function is probably called with a
> > struct ip * somewhere.
> 
> yes, that's what I think may need rewriting.

Ah, grand...

	David.
Received on Tue Jul 06 2004 - 09:48:20 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:38:00 UTC