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