Source-Changes archive

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

Re: CVS commit: src/usr.sbin/config



Fri, Jun 27, 2003 at 10:03:05AM +0900, enami tsugutomo:
> > I do not understand what you're asking.  the cast?
> > 
> >  undo_option(struct hashtab *ht, struct nvlist **npp,
> > +    struct nvlist ***next, const char *name, const char *type)
> > 
> > +               if (next != NULL && *next == (struct nvlist **) *npp)
> > +                       *next = npp;
> 
> Yes.  It doesn't work if nv_next is not a first member of the struct.
> &(*npp)->nv_next will be fine and makes code easier to read.

ah, now I understand.  quite correct.  Thanks enami!



Home | Main Index | Thread Index | Old Index