From owner-cvs-all Thu Aug 26 10:12:11 1999 Delivered-To: cvs-all@freebsd.org Received: from critter.freebsd.dk (dot2.cybercity.dk [212.242.41.138]) by hub.freebsd.org (Postfix) with ESMTP id 8F3C215E07; Thu, 26 Aug 1999 10:11:43 -0700 (PDT) (envelope-from phk@critter.freebsd.dk) Received: from critter.freebsd.dk (localhost [127.0.0.1]) by critter.freebsd.dk (8.9.3/8.9.2) with ESMTP id TAA01213; Thu, 26 Aug 1999 19:10:19 +0200 (CEST) (envelope-from phk@critter.freebsd.dk) To: Mike Smith Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/isofs/cd9660 cd9660_vfsops.c src/sys/kern vfs_subr.c vfs_syscalls.c src/sys/miscfs/devfs devfs_tree.c src/sys/miscfs/specfs spec_vnops.c src/sys/nfs nfs_subs.c src/sys/svr4 svr4_fcntl.c src/sys/sys conf.h lock.h vnode.h ... In-reply-to: Your message of "Thu, 26 Aug 1999 09:48:36 PDT." <199908261648.JAA02354@dingo.cdrom.com> Date: Thu, 26 Aug 1999 19:10:18 +0200 Message-ID: <1211.935687418@critter.freebsd.dk> From: Poul-Henning Kamp Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk In message <199908261648.JAA02354@dingo.cdrom.com>, Mike Smith writes: >> Rename checkalias2() and checkalias() to addalias() and >> addaliasu() - which takes dev_t and udev_t arg respectively. > >To be consistent with eg. vput(), vcount() these should be vaddalias* >(and vn_* would be better again). > >There are a lot of macros and functions with totally opaque names >currently in use (eg. btodb); if you're on a namechanging rampage it's >always good to try to make them less so. 8) Good point. I'll remember if I need to tweak it again at some point. >(Yes, I believe that this sort of feedback is more appropriate than a > public review of patches like these.) All constructive feedback is welcome in my mailbox. -- Poul-Henning Kamp FreeBSD coreteam member phk@FreeBSD.ORG "Real hackers run -current on their laptop." FreeBSD -- It will take a long time before progress goes too far! To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message