forked from junit-team/junit4
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Issue junit-team#426 Added suggestions by dsaff
Removed the Markdown syntax, changed import order and removed confusing Maximum Blank Lines section.
- Loading branch information
Arjan Wulder
committed
Sep 13, 2012
1 parent
3dbb0b3
commit ab1eb66
Showing
2 changed files
with
138 additions
and
136 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,138 @@ | ||
================================== | ||
Coding style | ||
================================== | ||
|
||
---------------------------------- | ||
Tabs and Indents | ||
---------------------------------- | ||
* Tab size : 4 | ||
* Indent : 4 | ||
* Continuation indent : 8 | ||
* Label indent : 0 | ||
|
||
> Don't use tab characters. | ||
|
||
---------------------------------- | ||
Spaces | ||
---------------------------------- | ||
Before Parentheses | ||
* if parentheses | ||
* for parentheses | ||
* while parentheses | ||
* switch parentheses | ||
* try parentheses | ||
* catch parentheses | ||
* synchronized parentheses | ||
|
||
Around Operators | ||
* Assignment operators (=, +=, …) | ||
* Logical operators (&&, ||) | ||
* Equality operators (==, !=) | ||
* Relational operators (<, >, <=, >=) | ||
* Bitwise operators (&, |, ^) | ||
* Additive operators (+, -) | ||
* Multiplicative operators (*, /, %) | ||
* Shift operators (<<, >>, >>>) | ||
|
||
Before Left Brace | ||
* Class left brace | ||
* Method left brace | ||
* if left brace | ||
* else left brace | ||
* for left brace | ||
* while left brace | ||
* do left brace | ||
* switch left brace | ||
* try left brace | ||
* catch left brace | ||
* finally left brace | ||
* synchronized left brace | ||
|
||
Before Keywords | ||
* else keyword | ||
* while keyword | ||
* catch keyword | ||
* finally keyword | ||
|
||
In Ternary Operator (?:) | ||
* Before ? | ||
* After ? | ||
* Before : | ||
* After : | ||
|
||
Within Type Arguments | ||
* After comma | ||
|
||
Other | ||
* After comma | ||
* After semicolon | ||
* After type cast | ||
|
||
---------------------------------- | ||
Wrapping and Braces | ||
---------------------------------- | ||
Braces placement | ||
* In class declaration : End of line | ||
* In method declaration : End of line | ||
* Other : End of line | ||
|
||
Use Of Braces | ||
* if() statement : When multiline | ||
* for() statement : When multiline | ||
* while() statement : When multiline | ||
* do .. while() statement : When multiline | ||
|
||
Annotations | ||
* Class annotations : Wrap always | ||
* Method annotations : Wrap always | ||
* Field annotations : Wrap always | ||
* Paramater annotations : Do not wrap | ||
* Local variable annotations : Do not wrap | ||
|
||
---------------------------------- | ||
Blank Lines | ||
---------------------------------- | ||
Minimum Blank Lines | ||
* Before package statement : 0 | ||
* After package statement : 1 | ||
* Before imports : 1 | ||
* After imports : 1 | ||
* Around class : 1 | ||
* After class header : 0 | ||
* After anonymous class header : 0 | ||
* Around field in interface : 0 | ||
* Around field : 0 | ||
* Around method in interface : 1 | ||
* Around method : 1 | ||
* Before method body : 0 | ||
|
||
---------------------------------- | ||
JavaDoc | ||
---------------------------------- | ||
Alignment | ||
* Align parameter descriptions | ||
* Align thrown exception descriptions | ||
|
||
Blank Lines | ||
* After description | ||
|
||
Invalid tags | ||
* Keep empty @param tags | ||
* Keep empty @return tags | ||
* Keep empty @throws tags | ||
|
||
Other | ||
* Enable leading asterisks | ||
* Use @throws rather then @exception | ||
* Keep empty lines | ||
|
||
---------------------------------- | ||
Imports | ||
---------------------------------- | ||
import static (all other imports) | ||
<blank line> | ||
import java.* | ||
import javax.* | ||
import com.* | ||
<blank line> | ||
import (all other imports) |
This file was deleted.
Oops, something went wrong.