You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by bu...@apache.org on 2016/07/25 15:24:36 UTC
svn commit: r993724 - in /websites/staging/jena/trunk/content: ./
getting_involved/reviewing_contributions.html
Author: buildbot
Date: Mon Jul 25 15:24:36 2016
New Revision: 993724
Log:
Staging update by buildbot for jena
Modified:
websites/staging/jena/trunk/content/ (props changed)
websites/staging/jena/trunk/content/getting_involved/reviewing_contributions.html
Propchange: websites/staging/jena/trunk/content/
------------------------------------------------------------------------------
--- cms:source-revision (original)
+++ cms:source-revision Mon Jul 25 15:24:36 2016
@@ -1 +1 @@
-1754022
+1754028
Modified: websites/staging/jena/trunk/content/getting_involved/reviewing_contributions.html
==============================================================================
--- websites/staging/jena/trunk/content/getting_involved/reviewing_contributions.html (original)
+++ websites/staging/jena/trunk/content/getting_involved/reviewing_contributions.html Mon Jul 25 15:24:36 2016
@@ -188,6 +188,7 @@ be removed by the committer who integrat
<h2 id="code-style">Code style<a class="headerlink" href="#code-style" title="Permanent link">¶</a></h2>
<p>Jena does not have a particular formal code style specification, but here are some simple tips for keeping your contribution in good order:</p>
<ul>
+<li>Jena uses the Java code conventions with spaces (not tabs!), an indent of 4, and opening braces on the same line. Use no trailing whitespace if avoidable. Use common sense to make your code readable for the next person.</li>
<li>Don't create a method signature that throws checked exceptions that aren't ever actually thrown from the code in that method unless an API supertype specifies that signature. Otherwise, clients of your code will have to include unnecessary handling code.</li>
<li>Don't leave unused imports in your code. IDEs provide facilities to clean imports.</li>
<li>If a type declares a supertype that isn't a required declaration, consider whether that clarifies or confuses the intent.</li>