DragonFly BSD
DragonFly commits List (threaded) for 2005-08
[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]

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


From: Matthew Dillon <dillon@xxxxxxxxxxxxxxxxxxxx>
Date: Wed, 3 Aug 2005 08:21:46 -0700 (PDT)

:
: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@xxxxxxxxxxxxx>



[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]