Re: HEADS UP Re: cvs commit: src/crypto/openssh/pam_ssh pam_ssh.csrc/gnu/usr.bin/binutils/gdb freebsd-uthread.c src/include mpool.hsrc/lib/libc/net name6.c src/lib/libc_r/uthread pthread_private.h uthread_file.csrc/lib/libncp ncpl_rcfile.c src/lib/libstand if_ether.h ...

[Bruce Evans]

On Thu, 25 May 2000, Poul-Henning Kamp wrote:

> In message <[EMAIL PROTECTED]>, Bruce Evans
>  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 couldn't (usefully) be a union, because then there would be no space
in the object for more than the list pointers.

It would be a style bug for it to be a class, since if you have classes
then you have C++ (or maybe objc) and then you should use C++ features
and not cpp macro hacks to implement queue.

It would be a style bug for it to be a typedef'ed struct type, since
struct types shouldn't be typedefed (see style(9)).

Bruce



To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message