> > > > 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".
Understood, but the point being that Garrett is an experienced
programmer, and made this mistake. The latter version avoids any
possibilities of said mistake.
Nate
To Unsubscribe: send mail to [email protected]
with "unsubscribe freebsd-current" in the body of the message
