On Wednesday 19 October 2011 21:07:11 Doug Anderson wrote: > On Wed, Oct 19, 2011 at 3:52 PM, Mike Frysinger <vap...@gentoo.org> wrote: > > i'm not particularly enamored with this naming style. i find the style > > of "<section>_<operation>" to be easier on the eyes rather than this > > RPN. > > > > i.e. cmdline_param_remove() and cmdline_param_add() > > I'm happy to name it whatever you'd like. My next patch will use your > naming suggestion unless someone says otherwise. > > I will note, however, that the "verb_noun" style of naming appears to be > favored by the Linux kernel. In the section of "Naming" in < > http://www.kernel.org/doc/Documentation/CodingStyle>, a suggested name is > "count_active_users()", not "active_users_count()".
to be fair, that file says:
... you should call that "count_active_users()" or similar, you should
_not_ call it "cntusr()".
it doesn't seem to take a position on the "verb_noun" vs "noun_verb" style.
in my experience, the common string tends to be more used at the start rather
than the end (spinlocks, mutexes, interrupts, gpios, spi, i2c). but i've
certainly not done a comprehensive survey to back up my position :).
-mike
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
