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

Justin T. Gibbs gibbs at FreeBSD.org
Thu Jun 9 17:25:20 UTC 2011


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?

--
Justin



More information about the svn-src-all mailing list