On Sat, 6 Mar 1999, Nate Williams wrote:
> > > I make a habit of checking out any interesting kernel changes in full,
> > > and one
> > > thing I found in the recent diff by Mr. Wollman for kern_timeout.c is:
> > >
> > > +void
> > > +callout_init(c)
> > > + struct callout *c;
> > > +{
> > > + bzero(c, sizeof c);
> > > }
> > >
> > > That doesn't look correct, does it?
> >
> > Agreed. I think it should be "sizeof *c".
>
> Ahh, I see. I think it should say
>
> bzero(c, sizeof(struct callout));
>
> To avoid the compiler using the size of the pointers et. al.
>
I prefer to do this in my own code, usually. However, using *c doesn't
"use the size of the pointer".
>
> Nate
>
>
> To Unsubscribe: send mail to [email protected]
> with "unsubscribe freebsd-current" in the body of the message
>
Brian Feldman _ __ ___ ___ ___
[email protected] _ __ ___ | _ ) __| \
http://www.freebsd.org/ _ __ ___ ____ | _ \__ \ |) |
FreeBSD: The Power to Serve! _ __ ___ ____ _____ |___/___/___/
To Unsubscribe: send mail to [email protected]
with "unsubscribe freebsd-current" in the body of the message
