Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 26 May 2000 08:19:40 -0700
From:      Jake Burkholder <jburkhol@home.com>
To:        Poul-Henning Kamp <phk@critter.freebsd.dk>
Cc:        Mike Smith <msmith@FreeBSD.ORG>, FreeBSD-current <current@FreeBSD.ORG>
Subject:   Re: HEADS UP Re: cvs commit: src/crypto/openssh/pam_ssh pam_ssh.c  src/gnu/usr.bin/binutils/gdb freebsd-uthread.c src/include mpool.h  src/lib/libc/net name6.c src/lib/libc_r/uthread pthread_private.h  uthread_file.c src/lib/libncp ncpl_rcfile.c src/lib/libstand if_ether.h ...
Message-ID:  <20000526151940.6B67CBA69@io.yi.org>
In-Reply-To: Message from Poul-Henning Kamp <phk@critter.freebsd.dk>  of "Fri, 26 May 2000 08:38:20 %2B0200." <9833.959323100@critter.freebsd.dk> 

next in thread | previous in thread | raw e-mail | index | archive | help
> In message <200005252326.QAA02413@mass.cdrom.com>, Mike Smith writes:
> >> >I objected to a recent commit hiding the fact that this is
> >> >"(elm)->field.sle_next".  Anyway, curelm must be a pointer to a struct.
> >> >Not just any struct; the struct must contain a "field" declared using
> >> >SLIST_ENTRY().
> >> 
> >> It could be an union or class as well...
> >
> >It would not be very useful if it were a union; the class issue is valid 
> >(although you could trivially use a struct contained within a class and a 
> >parent reference) but definitely not a good enough argument to support 
> >the massive breakage this otherwise entails.
> 
> I have yet to see any signs of "massive breakage".

Please just let it rest.

I don't think Mike's comments are out of line; this was massive
breakage regardless of wether world built or not.



To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20000526151940.6B67CBA69>