Subject: re: CVS commit: src/sys/kern
To: Simon Burge <simonb@NetBSD.org>
From: matthew green <mrg@eterna.com.au>
List: source-changes
Date: 09/10/2006 17:14:38
   
   > simonb@NetBSD.org wrote:
   > 
   > > > splclock()/splx() unmatch?
   > > 
   > > Yes indeed.  I was a bit hasty commiting that.
   > > 
   > > Please commit your fix.
   > 
   > Some people would hate goto (though I think it's okay for "break" case).
   > Which is better?
   
   In general I don't like goto either, but really I think it's the
   cleanest way for dealing with these error cases.

i agree.  it's also very common in error recovery in our kernel and
only slightly less so in userland..


.mrg.