Changeset 18514 in josm for trunk/tools

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

Location:

trunk/tools

Files:

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

trunk/tools/ivy.xml

@@ -15 +15 @@
     <dependencies>
         <!-- javacc->default -->
-        <dependency org="net.java.dev.javacc" name="javacc" rev="7.0.11" conf="javacc->default"/>
+        <dependency org="net.java.dev.javacc" name="javacc" rev="7.0.12" conf="javacc->default"/>
         <!-- checkstyle->default -->
         <dependency org="com.puppycrawl.tools" name="checkstyle" rev="9.3" conf="checkstyle->default"/>
         <!-- proguard->default -->
-        <dependency org="com.guardsquare" name="proguard-ant" rev="7.2.1" conf="proguard->default"/>
+        <dependency org="com.guardsquare" name="proguard-ant" rev="7.2.2" conf="proguard->default"/>
         <!-- pmd->default -->
-        <dependency org="net.sourceforge.pmd" name="pmd-core" rev="6.44.0" conf="pmd->default"/>
-        <dependency org="net.sourceforge.pmd" name="pmd-java" rev="6.44.0" conf="pmd->default"/>
+        <dependency org="net.sourceforge.pmd" name="pmd-core" rev="6.47.0" conf="pmd->default"/>
+        <dependency org="net.sourceforge.pmd" name="pmd-java" rev="6.47.0" conf="pmd->default"/>
         <dependency org="net.sourceforge.saxon" name="saxon" rev="9.1.0.8" conf="pmd->default">
             <artifact name="saxon" type="jar"/>
@@ -28 +28 @@
         </dependency>
         <!-- spotbugs->default -->
-        <dependency org="com.github.spotbugs" name="spotbugs" rev="4.6.0" conf="spotbugs->default"/>
-        <dependency org="com.github.spotbugs" name="spotbugs-ant" rev="4.6.0" conf="spotbugs->default"/>
+        <dependency org="com.github.spotbugs" name="spotbugs" rev="4.7.1" conf="spotbugs->default"/>
+        <dependency org="com.github.spotbugs" name="spotbugs-ant" rev="4.7.1" conf="spotbugs->default"/>
         <!-- errorprone->default -->
         <dependency org="com.google.errorprone" name="error_prone_core" rev="2.10.0" conf="errorprone->default"/>

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"/>