You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2022/09/15 08:20:56 UTC
[tomcat-jakartaee-migration] 03/03: Fix Checkstyle errors
This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tomcat-jakartaee-migration.git
commit 2c0552ab668100dfb5677c366d43f64f49470d5d
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Thu Sep 15 09:20:47 2022 +0100
Fix Checkstyle errors
---
src/main/java/org/apache/tomcat/jakartaee/AntHandler.java | 4 ++--
.../java/org/apache/tomcat/jakartaee/ClassConverter.java | 1 +
.../java/org/apache/tomcat/jakartaee/EESpecProfile.java | 10 +++++-----
src/main/java/org/apache/tomcat/jakartaee/GlobMatcher.java | 8 ++------
src/main/java/org/apache/tomcat/jakartaee/Migration.java | 13 +++++++++----
5 files changed, 19 insertions(+), 17 deletions(-)
diff --git a/src/main/java/org/apache/tomcat/jakartaee/AntHandler.java b/src/main/java/org/apache/tomcat/jakartaee/AntHandler.java
index e7747b7..5946226 100644
--- a/src/main/java/org/apache/tomcat/jakartaee/AntHandler.java
+++ b/src/main/java/org/apache/tomcat/jakartaee/AntHandler.java
@@ -27,11 +27,11 @@ import org.apache.tools.ant.Task;
/**
* JUL log handler redirecting the messages logged to Ant.
*/
- class AntHandler extends Handler {
+class AntHandler extends Handler {
private final Task task;
- public AntHandler(Task task) {
+ AntHandler(Task task) {
this.task = task;
}
diff --git a/src/main/java/org/apache/tomcat/jakartaee/ClassConverter.java b/src/main/java/org/apache/tomcat/jakartaee/ClassConverter.java
index da9ee42..8b4116d 100644
--- a/src/main/java/org/apache/tomcat/jakartaee/ClassConverter.java
+++ b/src/main/java/org/apache/tomcat/jakartaee/ClassConverter.java
@@ -26,6 +26,7 @@ import java.lang.instrument.IllegalClassFormatException;
import java.security.ProtectionDomain;
import java.util.logging.Level;
import java.util.logging.Logger;
+
import org.apache.bcel.classfile.ClassParser;
import org.apache.bcel.classfile.Constant;
import org.apache.bcel.classfile.ConstantUtf8;
diff --git a/src/main/java/org/apache/tomcat/jakartaee/EESpecProfile.java b/src/main/java/org/apache/tomcat/jakartaee/EESpecProfile.java
index 23eecbc..8ed0d91 100644
--- a/src/main/java/org/apache/tomcat/jakartaee/EESpecProfile.java
+++ b/src/main/java/org/apache/tomcat/jakartaee/EESpecProfile.java
@@ -24,7 +24,7 @@ import java.util.regex.Pattern;
*/
public enum EESpecProfile {
- TOMCAT("javax", "jakarta",
+ TOMCAT("javax", "jakarta",
"javax([/\\.](annotation(?![/\\.]processing)" +
"|ejb" +
"|el" +
@@ -35,7 +35,7 @@ public enum EESpecProfile {
"|transaction(?![/\\.]xa)" +
"|websocket))"),
- EE("javax", "jakarta",
+ EE("javax", "jakarta",
"javax([/\\.](activation" +
"|annotation(?![/\\.]processing)" +
"|batch" +
@@ -60,7 +60,7 @@ public enum EESpecProfile {
"|websocket" +
"|ws[/\\.]rs" +
"|xml[/\\.](bind|soap|ws)))"),
- JEE8("jakarta", "javax",
+ JEE8("jakarta", "javax",
"jakarta([/\\.](activation" +
"|annotation(?![/\\.]processing)" +
"|batch" +
@@ -100,12 +100,12 @@ public enum EESpecProfile {
Matcher m = pattern.matcher(name);
return m.replaceAll(target + "$1");
}
-
+
public String getSource()
{
return source;
}
-
+
public String getTarget()
{
return target;
diff --git a/src/main/java/org/apache/tomcat/jakartaee/GlobMatcher.java b/src/main/java/org/apache/tomcat/jakartaee/GlobMatcher.java
index 33dc338..87bff59 100644
--- a/src/main/java/org/apache/tomcat/jakartaee/GlobMatcher.java
+++ b/src/main/java/org/apache/tomcat/jakartaee/GlobMatcher.java
@@ -233,11 +233,7 @@ public final class GlobMatcher {
return true;
}
- private static boolean different(
- boolean caseSensitive, char ch, char other) {
- return caseSensitive
- ? ch != other
- : Character.toUpperCase(ch) != Character.toUpperCase(other);
+ private static boolean different(boolean caseSensitive, char ch, char other) {
+ return caseSensitive ? ch != other : Character.toUpperCase(ch) != Character.toUpperCase(other);
}
-
}
diff --git a/src/main/java/org/apache/tomcat/jakartaee/Migration.java b/src/main/java/org/apache/tomcat/jakartaee/Migration.java
index d8b899c..35bfcbe 100644
--- a/src/main/java/org/apache/tomcat/jakartaee/Migration.java
+++ b/src/main/java/org/apache/tomcat/jakartaee/Migration.java
@@ -195,7 +195,7 @@ public class Migration {
boolean inplace = src.equals(dest);
if (!inplace) {
try (InputStream is = new FileInputStream(src);
- OutputStream os = new FileOutputStream(dest)) {
+ OutputStream os = new FileOutputStream(dest)) {
migrateStream(src.getName(), is, os);
}
} else {
@@ -283,8 +283,12 @@ public class Migration {
private boolean isSignatureFile(String sourceName) {
- return sourceName.startsWith("META-INF/")
- && (sourceName.endsWith(".SF") || sourceName.endsWith(".RSA") || sourceName.endsWith(".DSA") || sourceName.endsWith(".EC"));
+ return sourceName.startsWith("META-INF/") && (
+ sourceName.endsWith(".SF") ||
+ sourceName.endsWith(".RSA") ||
+ sourceName.endsWith(".DSA") ||
+ sourceName.endsWith(".EC")
+ );
}
@@ -334,7 +338,8 @@ public class Migration {
}
private static class MigrationZipArchiveEntry extends ZipArchiveEntry {
- public MigrationZipArchiveEntry(ZipArchiveEntry entry) throws ZipException {
+
+ MigrationZipArchiveEntry(ZipArchiveEntry entry) throws ZipException {
super(entry);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org