Date: Sun, 12 Jan 2003 09:59:35 +0100 From: phk@freebsd.org To: Matt Dillon <dillon@freebsd.org> Cc: cvs-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/kern kern_acl.c kern_descrip.c kern_event.c kern_mac.c sys_pipe.c sys_socket.c uipc_socket.c uipc_syscalls.c uipc_usrreq.c vfs_aio.c vfs_syscalls.c vfs_vnops.c src/sys/netgraph ng_socket.c src/sys/sys file.h src/sys/ufs/ffs ... Message-ID: <3902.1042361975@critter.freebsd.dk> In-Reply-To: Your message of "Sat, 11 Jan 2003 17:37:13 PST." <200301120137.h0C1bD0E098037@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
In message <200301120137.h0C1bD0E098037@repoman.freebsd.org>, Matt Dillon write s: > sys/sys file.h > Log: > Change struct file f_data to un_data, a union of the correct struct > pointer types, and remove a huge number of casts from code using it. This change is in the wrong direction: Rather than keep the layering strict, it ruins the layering by not almost but not quite teaching struct file about all the underlying kinds of stuff. With this change we have to modify sys/file.h if we add a new backing for struct file, something which had until now not been necessary. This cost is not offset by the fact that the source code is not any clearer and mostly it hides the simplicity of the object model used. (in the case of sys/file.h it's worse). You could instead just have removed all the unneeded casts, (there may be one or two legitimate ones), but even that is a bad idea at a time where so many people diff between RELENG_5_0 and -current. Please back it out. Poul-Henning -- Poul-Henning Kamp | UNIX since Zilog Zeus 3.20 phk@FreeBSD.ORG | TCP/IP since RFC 956 FreeBSD committer | BSD since 4.3-tahoe Never attribute to malice what can adequately be explained by incompetence. To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?3902.1042361975>