Changeset 18514 in josm for trunk/tools/pmd

Timestamp:

2022-07-12T17:16:14+02:00 (3 years ago)

Author:

taylor.smock

Message:

Dependency updates

ivy.xml:

• metadata-extractor: 2.17.0 -> 2.18.0
  • Full changelog: ​https://github.com/drewnoakes/metadata-extractor/releases/tag/2.18.0
• spotbugs: 4.6.0 -> 4.7.1
  • New detector ThrowingExceptions which introduces checks for methods throwing RuntimeException, Exception, or Throwable
  • Full changelog: ​https://github.com/spotbugs/spotbugs/releases/tag/4.7.0 (4.7.1 was bugfixes)
• wiremock-jre8: 2.33.1 -> 2.33.2
  • 2.33.1 was built with Java 11 instead of Java 8, leading to some incompatibilities
• classgraph: 4.8.146 -> 4.8.149
  • Line number ranges for methods
  • Several bug fixes

tools/ivy.xml

• javacc: 7.0.11 -> 7.0.12
  • Mostly formatting fixes for generated code
• proguard: 7.2.1 -> 7.2.2
  • Full changelog: ​https://github.com/Guardsquare/proguard/releases/tag/v7.2.2

pmd: 6.44.0 -> 6.47.0

• EmptyControlStatement (we have a few empty if blocks) This replaces EmptyFinallyBlock, EmptyIfStmt, EmptyInitializer, EmptyStatementBlock, EmptySwitchStatements, EmptySynchronizedBlock, EmptyTryBlock, and EmptyWhileStmt We currently ignore this, since we were previously ignoring EmptyIfStmt and EmptyWhileStmt

File:

• trunk/tools/pmd/josm-ruleset.xml (modified) (2 diffs)

trunk/tools/pmd/josm-ruleset.xml

@@ -57 +57 @@
     <exclude name="ControlStatementBraces"/>
     <exclude name="DefaultPackage"/>
+    <!-- EmptyControlStatement is not configurable, so the behavior of the old excludes of EmptyIfStmt/EmptyWhileStmt (errorprone.xml) while getting new functionality cannot be done -->
+    <exclude name="EmptyControlStatement"/>
     <exclude name="EmptyMethodInAbstractClassShouldBeAbstract"/>
     <exclude name="FieldDeclarationsShouldBeAtStartOfClass"/>
@@ -175 +177 @@
     <exclude name="DoNotTerminateVM"/>
     <exclude name="DontImportSun"/>
-    <exclude name="EmptyIfStmt"/>
-    <exclude name="EmptyWhileStmt"/>
     <exclude name="ImplicitSwitchFallThrough"/>
     <exclude name="MissingSerialVersionUID"/>