Subject: Re: CVS commit: src/usr.sbin/config
To: enami tsugutomo <enami@sm.sony.co.jp>
From: john heasley <heas@shrubbery.net>
List: source-changes
Date: 06/26/2003 21:43:17
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!