Pmd clean up (JSQLParser#1215)

* Add PMD Annotations in order to avoid useless exceptions for the Deparsers

* Add Eclipse Formatter configuration

* Fix typo

* Replace Comments on empty methods with Class wide PMD Annotation
Do not enforce checkstyle formatting

Author: manticore-projects

README.md

@@ -63,7 +63,7 @@ Also I would like to know about needed examples or documentation stuff.
 ** API change FunctionWithCondParams production removed
 * API change in ValuesStatement: the expression list is now hold as a ItemList and not as a List<Expression>
 * support for parser modification within **parseExpression** and **parseCondExpression**
-' support for table schema for foreign keys
+* support for table schema for foreign keys
 * support for Oracle hints on **insert, update and merge**
 * support for **merge insert where** clause
 * allow **in** as schema name

eclipse-java-google-style.xml

@@ -20,5 +20,13 @@
         <com-junichi11-netbeans-changelf.use-global>false</com-junichi11-netbeans-changelf.use-global>
         <org-netbeans-modules-javascript2-requirejs.enabled>true</org-netbeans-modules-javascript2-requirejs.enabled>
         <netbeans.hint.jdkPlatform>JDK_1.8</netbeans.hint.jdkPlatform>
+        <org-netbeans-modules-editor-indent.text.xml.CodeStyle.project.expand-tabs>false</org-netbeans-modules-editor-indent.text.xml.CodeStyle.project.expand-tabs>
+        <org-netbeans-modules-editor-indent.CodeStyle.project.text-line-wrap>none</org-netbeans-modules-editor-indent.CodeStyle.project.text-line-wrap>
+        <org-netbeans-modules-editor-indent.CodeStyle.project.indent-shift-width>4</org-netbeans-modules-editor-indent.CodeStyle.project.indent-shift-width>
+        <org-netbeans-modules-editor-indent.CodeStyle.project.spaces-per-tab>4</org-netbeans-modules-editor-indent.CodeStyle.project.spaces-per-tab>
+        <org-netbeans-modules-editor-indent.CodeStyle.project.tab-size>4</org-netbeans-modules-editor-indent.CodeStyle.project.tab-size>
+        <org-netbeans-modules-editor-indent.CodeStyle.project.text-limit-width>120</org-netbeans-modules-editor-indent.CodeStyle.project.text-limit-width>
+        <org-netbeans-modules-editor-indent.CodeStyle.project.expand-tabs>true</org-netbeans-modules-editor-indent.CodeStyle.project.expand-tabs>
+        <org-netbeans-modules-editor-indent.CodeStyle.usedProfile>project</org-netbeans-modules-editor-indent.CodeStyle.usedProfile>
     </properties>
 </project-shared-configuration>

nb-configuration.xml

@@ -110,5 +110,7 @@ under the License.
 
     <rule ref="category/java/performance.xml/BigIntegerInstantiation" />
     <rule ref="category/java/performance.xml/BooleanInstantiation" />
-
+    
+    <!-- for Codazy -->
+    <rule ref="category/java/documentation.xml/UncommentedEmptyMethodBody" />
 </ruleset>

pmd-rules.xml

@@ -365,6 +365,13 @@
                 <artifactId>javacc-maven-plugin</artifactId>
                 <version>2.6</version>
             </plugin>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-checkstyle-plugin</artifactId>
+                <configuration>
+                    <configLocation>config/sun_checks.xml</configLocation>
+                </configuration>
+            </plugin>
         </plugins>
     </reporting>
 

pom.xml

@@ -179,7 +179,7 @@ public void setIgnoreNulls(boolean ignoreNulls) {
     }
 
     @Override
-    @SuppressWarnings({"PMD.CyclomaticComplexity", "PMD.ExcessiveMethodLength"})
+    @SuppressWarnings({"PMD.CyclomaticComplexity", "PMD.NPathComplexity"})
     public String toString() {
         StringBuilder b = new StringBuilder();
 

src/main/java/net/sf/jsqlparser/expression/AnalyticExpression.java

@@ -30,7 +30,7 @@
 import net.sf.jsqlparser.statement.select.UnPivot;
 import net.sf.jsqlparser.statement.select.WithItem;
 
-@SuppressWarnings({"PMD.CyclomaticComplexity"})
+@SuppressWarnings({"PMD.CyclomaticComplexity", "PMD.UncommentedEmptyMethodBody"})
 public class ExpressionVisitorAdapter implements ExpressionVisitor, ItemsListVisitor, PivotVisitor, SelectItemVisitor {
 
     private SelectVisitor selectVisitor;

src/main/java/net/sf/jsqlparser/expression/ExpressionVisitorAdapter.java

@@ -168,7 +168,7 @@ public void setKeep(KeepExpression keep) {
     }
 
     @Override
-    @SuppressWarnings({"PMD.CyclomaticComplexity"})
+    @SuppressWarnings({"PMD.CyclomaticComplexity", "PMD.NPathComplexity"})
     public String toString() {
         String params;
 

src/main/java/net/sf/jsqlparser/expression/Function.java

@@ -11,6 +11,7 @@
 
 import net.sf.jsqlparser.statement.select.SubSelect;
 
+@SuppressWarnings({"PMD.UncommentedEmptyMethodBody"})
 public class ItemsListVisitorAdapter implements ItemsListVisitor {
 
     @Override

src/main/java/net/sf/jsqlparser/expression/operators/relational/ItemsListVisitorAdapter.java

@@ -33,6 +33,7 @@
 import net.sf.jsqlparser.statement.upsert.Upsert;
 import net.sf.jsqlparser.statement.values.ValuesStatement;
 
+@SuppressWarnings({"PMD.UncommentedEmptyMethodBody"})
 public class StatementVisitorAdapter implements StatementVisitor {
 
     @Override

src/main/java/net/sf/jsqlparser/statement/StatementVisitorAdapter.java

@@ -360,7 +360,7 @@ public void setUk(boolean uk) {
   }
 
   @Override
-  @SuppressWarnings({"PMD.CyclomaticComplexity"})
+  @SuppressWarnings({"PMD.CyclomaticComplexity", "PMD.NPathComplexity"})
   public String toString() {
 
     StringBuilder b = new StringBuilder();

src/main/java/net/sf/jsqlparser/statement/alter/AlterExpression.java

@@ -141,7 +141,7 @@ public void setRowMovement(RowMovement rowMovement) {
     }
 
     @Override
-    @SuppressWarnings({"PMD.CyclomaticComplexity"})
+    @SuppressWarnings({"PMD.CyclomaticComplexity", "PMD.NPathComplexity"})
     public String toString() {
         String sql;
         String createOps = PlainSelect.getStringList(createOptionsStrings, false, false);

src/main/java/net/sf/jsqlparser/statement/create/table/CreateTable.java

@@ -207,7 +207,7 @@ public void setWithItemsList(List<WithItem> withItemsList) {
     }
 
     @Override
-    @SuppressWarnings({"PMD.CyclomaticComplexity"})
+    @SuppressWarnings({"PMD.CyclomaticComplexity", "PMD.NPathComplexity"})
     public String toString() {
         StringBuilder sql = new StringBuilder();
         if (withItemsList != null && !withItemsList.isEmpty()) {

src/main/java/net/sf/jsqlparser/statement/insert/Insert.java

@@ -11,6 +11,7 @@
 
 import net.sf.jsqlparser.schema.Table;
 
+@SuppressWarnings({"PMD.UncommentedEmptyMethodBody"})
 public class FromItemVisitorAdapter implements FromItemVisitor {
 
     @Override

src/main/java/net/sf/jsqlparser/statement/select/FromItemVisitorAdapter.java

@@ -55,6 +55,7 @@ public void addGroupingSet(ExpressionList list) {
     }
 
     @Override
+    @SuppressWarnings({"PMD.CyclomaticComplexity"})
     public String toString() {
         StringBuilder b = new StringBuilder();
         b.append("GROUP BY ");

src/main/java/net/sf/jsqlparser/statement/select/GroupByElement.java

@@ -11,6 +11,7 @@
 
 import net.sf.jsqlparser.schema.Table;
 
+@SuppressWarnings({"PMD.UncommentedEmptyMethodBody"})
 public class IntoTableVisitorAdapter implements IntoTableVisitor {
 
     @Override

src/main/java/net/sf/jsqlparser/statement/select/IntoTableVisitorAdapter.java

@@ -263,6 +263,7 @@ public boolean isWindowJoin() {
 
     /**
      * Return the "WITHIN" join window (if any)
+     * @return 
      */
     public KSQLJoinWindow getJoinWindow() {
         return joinWindow;
@@ -278,7 +279,7 @@ public void setJoinWindow(KSQLJoinWindow joinWindow) {
     }
 
     @Override
-    @SuppressWarnings({"PMD.CyclomaticComplexity"})
+    @SuppressWarnings({"PMD.CyclomaticComplexity", "PMD.NPathComplexity"})
     public String toString() {
         if (isSimple() && isOuter()) {
             return "OUTER " + rightItem;

src/main/java/net/sf/jsqlparser/statement/select/Join.java

@@ -9,6 +9,7 @@
  */
 package net.sf.jsqlparser.statement.select;
 
+@SuppressWarnings({"PMD.UncommentedEmptyMethodBody"})
 public class OrderByVisitorAdapter implements OrderByVisitor {
 
     @Override

src/main/java/net/sf/jsqlparser/statement/select/OrderByVisitorAdapter.java

@@ -9,6 +9,7 @@
  */
 package net.sf.jsqlparser.statement.select;
 
+@SuppressWarnings({"PMD.UncommentedEmptyMethodBody"})
 public class PivotVisitorAdapter implements PivotVisitor {
 
     @Override

src/main/java/net/sf/jsqlparser/statement/select/PivotVisitorAdapter.java

@@ -322,7 +322,7 @@ public void setKsqlWindow(KSQLWindow ksqlWindow) {
     }
 
     @Override
-    @SuppressWarnings({"PMD.CyclomaticComplexity", "PMD.ExcessiveMethodLength"})
+    @SuppressWarnings({"PMD.CyclomaticComplexity" , "PMD.ExcessiveMethodLength", "PMD.NPathComplexity"})
     public String toString() {
         StringBuilder sql = new StringBuilder();
         if (useBrackets) {

src/main/java/net/sf/jsqlparser/statement/select/PlainSelect.java

@@ -42,6 +42,7 @@ public void setSelectBody(SelectBody body) {
     }
 
     @Override
+    @SuppressWarnings({"PMD.CyclomaticComplexity", "PMD.NPathComplexity"})
     public String toString() {
         StringBuilder retval = new StringBuilder();
         if (withItemsList != null && !withItemsList.isEmpty()) {

src/main/java/net/sf/jsqlparser/statement/select/Select.java

@@ -9,6 +9,7 @@
  */
 package net.sf.jsqlparser.statement.select;
 
+@SuppressWarnings({"PMD.UncommentedEmptyMethodBody"})
 public class SelectItemVisitorAdapter implements SelectItemVisitor {
 
     @Override

src/main/java/net/sf/jsqlparser/statement/select/SelectItemVisitorAdapter.java

@@ -11,6 +11,7 @@
 
 import net.sf.jsqlparser.statement.values.ValuesStatement;
 
+@SuppressWarnings({"PMD.UncommentedEmptyMethodBody"})
 public class SelectVisitorAdapter implements SelectVisitor {
 
     @Override

src/main/java/net/sf/jsqlparser/statement/select/SelectVisitorAdapter.java

@@ -12,6 +12,7 @@
 import net.sf.jsqlparser.expression.Alias;
 import net.sf.jsqlparser.expression.Function;
 
+@SuppressWarnings({"PMD.UncommentedEmptyMethodBody"})
 public class TableFunction extends FunctionItem implements FromItem {
 
     @Override

src/main/java/net/sf/jsqlparser/statement/select/TableFunction.java

@@ -19,6 +19,7 @@
 import net.sf.jsqlparser.expression.operators.relational.ExpressionList;
 import net.sf.jsqlparser.expression.operators.relational.MultiExpressionList;
 
+@SuppressWarnings({"PMD.UncommentedEmptyMethodBody"})
 public class ValuesList implements FromItem {
 
     private Alias alias;

src/main/java/net/sf/jsqlparser/statement/select/ValuesList.java

@@ -60,6 +60,7 @@ public void setWithItemList(List<SelectItem> withItemList) {
     }
 
     @Override
+    @SuppressWarnings({"PMD.CyclomaticComplexity"})
     public String toString() {
         return (recursive ? "RECURSIVE " : "") + name + ((withItemList != null) ? " " + PlainSelect.
                 getStringList(withItemList, true, true) : "")

src/main/java/net/sf/jsqlparser/statement/select/WithItem.java

@@ -21,6 +21,7 @@
  *
  * @author tw
  */
+@SuppressWarnings({"PMD.UncommentedEmptyMethodBody"})
 public abstract class ConnectExpressionsVisitor implements SelectVisitor, SelectItemVisitor {
 
     private String alias = "expr";