cvs commit: src/sys/kern vfs_bio.c vfs_syscalls.c src/sys/sys buf.h src/sys/vm vm_pager.c

Matthew Dillon dillon at apollo.backplane.com
Wed Aug 3 08:22:14 PDT 2005


:
:On Tue, Aug 02, 2005 at 09:59:53PM -0700, Hiten Pandya wrote:
:>   Localise buffer queue information into kern/vfs_bio.c, it should not be
:>   messed with outside of the named file.  Convert the QUEUE_* #defines
:>   into enum bufq_type, prefix the names with 'B'.  The change to initpbuf()
:>   is acceptable since they are a hack anyway, not to mention that
:
:You might want to change the various switch statements to have no
:default case, allowing GCC to warn about missing cases.
:
:Joerg
    
    I'd rather not depend on that feature of GCC, it could bite us in some future
    release of GCC, plus it is not always desireable to actually list all cases
    in a switch state (though in the buffer cache case its probably fine).
    A default/panic case is more robust.

					-Matt
					Matthew Dillon 
					<dillon at xxxxxxxxxxxxx>





More information about the Commits mailing list