svn commit: r221397 - in head/sys: kern sys

John Baldwin jhb at freebsd.org
Thu Jun 9 18:40:37 UTC 2011


On Thursday, June 09, 2011 1:25:13 pm Justin T. Gibbs wrote:
> On 5/3/11 12:54 PM, Andrey V. Elsukov wrote:
> >  Author: ae
> >  Date: Tue May 3 18:54:18 2011
> >  New Revision: 221397
> >  URL: http://svn.freebsd.org/changeset/base/221397
> >
> >  Log:
> >  Add make_dev_alias_p() function. It is similar to make_dev_alias(),
> >  but it may return an error like make_dev_p() does.
> 
> Sorry for the late reply on this change.  I noticed it during some recent
> merge activity at work where my make_dev_credv() conflicted with your
> make_dev_cred_v().
> 
> It seems that the existing convention in this file for "function variants"
> is to append a single letter to the function name without an underscore
> (e.g. make_dev_credv, make_dev_credf).  make_dev_alias_v() and make_dev_p(),
> break this convention.  It would be nice to clean up this API so that
> the naming conventions are consistent.  So which should it be?  Underscore
> or no underscore?

Hmm, could we take advantage of 9 being an API "breaking point" to simplify
the API back down so there are fewer variants perhaps?

Maybe we could make the 'flags' argument to make_dev() mandatory and change
make_dev and make_dev_alias to always use the _p semantics (return an error
and use a parameter to optionally return the new cdev)?

-- 
John Baldwin


More information about the svn-src-all mailing list