cvs commit: src/sys/dev/ata ata-all.c

M. Warner Losh imp at bsdimp.com
Tue Apr 22 00:02:31 UTC 2008


In message: <99531.1208814953 at critter.freebsd.dk>
            "Poul-Henning Kamp" <phk at phk.freebsd.dk> writes:
: In message <20080421213724.GL82555 at funkthat.com>, John-Mark Gurney writes:
: 
: >Any one object to changing:
: >.Nm devclass_get_maxunit
: >.Nd find the maximum unit number in the class
: >
: >to:
: >.Nm devclass_get_maxunit
: >.Nd find the next free unit number in the class
: 
: Those are very different semantics, the name should change as well
: then.

The change isn't in semantics of the function call.  It is just the
description of what it does.

Agreed that it is a bad interface name..

Warner


More information about the cvs-src mailing list