Fixups from review (#144)

adangel · adangel · commit 4aaf0da7f833 · 2024-04-08T19:57:55.000+02:00
diff --git a/src/it/MPMD-379-JDK21/invoker.properties b/src/it/MPMD-379-JDK21/invoker.properties
@@ -15,8 +15,6 @@
 # specific language governing permissions and limitations
 # under the License.
 
-invoker.java.version = 1.8+
-
 # available toolchains under linux:
 # https://github.com/apache/infrastructure-p6/blob/production/modules/build_nodes/files/toolchains.xml
 
diff --git a/src/main/java/org/apache/maven/plugins/pmd/exec/CpdExecutor.java b/src/main/java/org/apache/maven/plugins/pmd/exec/CpdExecutor.java
@@ -181,11 +181,11 @@ private CpdResult run() throws MavenReportException {
                         writeFormattedReport(report);
                     }
                 } catch (MavenReportException e) {
-                    LOG.error(e.getMessage(), e);
+                    LOG.error("Error while writing CPD report", e);
                 }
             });
         } catch (IOException e) {
-            LOG.error("Error while executing CPD: {}", e.getMessage(), e);
+            LOG.error("Error while executing CPD", e);
         }
         LOG.debug("CPD finished.");
 

Original file line number	Diff line number	Diff line change
`@@ -181,11 +181,11 @@ private CpdResult run() throws MavenReportException {`
`181`	`181`	`writeFormattedReport(report);`
`182`	`182`	`}`
`183`	`183`	`} catch (MavenReportException e) {`
`184`		`- LOG.error(e.getMessage(), e);`
	`184`	`+ LOG.error("Error while writing CPD report", e);`
`185`	`185`	`}`
`186`	`186`	`});`
`187`	`187`	`} catch (IOException e) {`
`188`		`- LOG.error("Error while executing CPD: {}", e.getMessage(), e);`
	`188`	`+ LOG.error("Error while executing CPD", e);`
`189`	`189`	`}`
`190`	`190`	`LOG.debug("CPD finished.");`
`191`	`191`