Re: CVS commit: src/sys/dev/scsipi

[James Chacon <jmc%NetBSD.org@localhost>]

On Fri, Dec 16, 2005 at 06:51:11PM +0100, Jaromir Dolecek wrote:
> On Fri, Dec 16, 2005 at 05:33:30PM +0000, Christos Zoulas wrote:
> > In article <87oe3hugp1.fsf%snark.piermont.com@localhost>,
> > Perry E. Metzger <perry%piermont.com@localhost> wrote:
> > >
> > >Reinoud Zandijk <reinoud%netbsd.org@localhost> writes:
> > >> Module Name:     src
> > >> Committed By:    reinoud
> > >> Date:            Thu Dec 15 17:56:32 UTC 2005
> > >>
> > >> Modified Files:
> > >>  src/sys/dev/scsipi: cd.c
> > >>
> > >> Log Message:
> > >> Clean up cdgetdisklabel. It was jumping over a return...
> > >
> > >Did anyone review this change?
> > 
> > Yes, it is fine.
> 
> Including the new non-optional printf()? Please back that part
> off.

It should probably be ifdef'd by DIAGNOSTIC.

James