diff options
author | Robin H. Johnson <robbat2@gentoo.org> | 2013-08-23 17:39:27 -0700 |
---|---|---|
committer | Robin H. Johnson <robbat2@gentoo.org> | 2013-08-23 17:51:09 -0700 |
commit | edfdd06aeecc784104e6eeba6ab227cae49c40d7 (patch) | |
tree | 26c55213e080e21678a42ebd894af2ab6dcca2d0 /STYLE | |
parent | rename package to netifrc (diff) | |
download | netifrc-edfdd06aeecc784104e6eeba6ab227cae49c40d7.tar.gz netifrc-edfdd06aeecc784104e6eeba6ab227cae49c40d7.tar.bz2 netifrc-edfdd06aeecc784104e6eeba6ab227cae49c40d7.zip |
We are sticking to the OpenRC style guide.
Signed-off-by: Robin H. Johnson <robbat2@gentoo.org>
Diffstat (limited to 'STYLE')
-rw-r--r-- | STYLE | 89 |
1 files changed, 89 insertions, 0 deletions
@@ -0,0 +1,89 @@ +This STYLE file taken directly from OpenRC, and based on the heritage of +netifrc, we are sticking to it. + +==== Original file beyond this point. +This is the openrc style manual. It governs the coding style of all code +in this repository. Follow it. Contact openrc@gentoo.org for any questions +or fixes you might notice. + +########## +# C CODE # +########## + +The BSD Kernel Normal Form (KNF) style is used: + http://en.wikipedia.org/wiki/Indent_style#BSD_KNF_style +Basically, it's like K&R/LKML, but wrapped lines that are indented use 4 spaces. + +Highlights: + - no trailing whitespace + - indented code use tabs (not line wrapped) + - cuddle the braces (except for functions) + - space after native statements and before paren (for/if/while/...) + - no space between function and paren + - pointer asterisk cuddles the variable, not the type + +void foo(int c) +{ + int ret = 0; + + if (c > 1000) + return; + + while (c--) { + bar(c); + ret++; + } + + return ret; +} + +################## +# COMMIT MESSAGES # +################## + +The following is an example of a correctly formatted git commit message +for this repository. Most of this information came from this blog post +[1], so I would like to thank the author. + +### cut here ### +Capitalized, short (50 chars or less) summary + +More detailed explanatory text, if necessary. Wrap it to about 72 +characters or so. In some contexts, the first line is treated as the +subject of an email and the rest of the text as the body. The blank +line separating the summary from the body is critical (unless you omit +the body entirely); tools like rebase can get confused if you run the +two together. + +Write your commit message in the imperative: "Fix bug" and not "Fixed +bug." This convention matches up with commit messages generated by +commands like git merge and git revert. + +Further paragraphs come after blank lines. + +- Bullet points are okay, too + +- Typically a hyphen or asterisk is used for the bullet, preceded by a + single space, with blank lines in between, but conventions vary here + +- Use a hanging indent + +Reported-by: User Name <email> +X-[Distro]-Bug: BugID +X-[Distro]-Bug-URL: URL for the bug (on the distribution's web site typically) +### cut here ### + +If you did not write the code and the patch does not include authorship +information in a format git can use, please use the --author option of the +git commit command to make the authorship correct. + +The Reported-by tag is required if the person who reported the bug is +different from the author and committer. + + The X-[Distro]-Bug/Bug-URL tags are required if this commit is related + to a bug reported to us by a specific distribution of linux or a + *BSD. Also, [Distro] should be replaced with the name of the + distribution, e.g. X-Gentoo-Bug. + +[1] http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html + |