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 2021/06/01 18:30:42 UTC
[tomcat] branch 10.0.x updated: Fix some indentation issues
This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch 10.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/10.0.x by this push:
new 8997577 Fix some indentation issues
8997577 is described below
commit 8997577ba0c5d60bd7ab01760545e3d834abbc1b
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Tue Jun 1 19:29:29 2021 +0100
Fix some indentation issues
Helps readability but mainly to trigger a CI build
---
java/jakarta/ejb/EJB.java | 14 +++++++-------
java/jakarta/el/BeanELResolver.java | 6 ++----
java/jakarta/el/ExpressionFactory.java | 3 +--
java/jakarta/el/ListELResolver.java | 3 +--
java/jakarta/el/Util.java | 20 ++++++++------------
java/jakarta/persistence/PersistenceContext.java | 10 +++++-----
java/jakarta/persistence/PersistenceProperty.java | 4 ++--
java/jakarta/persistence/PersistenceUnit.java | 4 ++--
res/checkstyle/checkstyle.xml | 2 +-
9 files changed, 29 insertions(+), 37 deletions(-)
diff --git a/java/jakarta/ejb/EJB.java b/java/jakarta/ejb/EJB.java
index 65f9e8f..82b9bfd 100644
--- a/java/jakarta/ejb/EJB.java
+++ b/java/jakarta/ejb/EJB.java
@@ -27,11 +27,11 @@ import java.lang.annotation.Target;
@Retention(RetentionPolicy.RUNTIME)
public @interface EJB {
- String name() default "";
- String description() default "";
- @SuppressWarnings("rawtypes") // Can't use Class<?> because API needs to match specification
- Class beanInterface() default java.lang.Object.class;
- String beanName() default "";
- String mappedName() default "";
- String lookup() default "";
+ String name() default "";
+ String description() default "";
+ @SuppressWarnings("rawtypes") // Can't use Class<?> because API needs to match specification
+ Class beanInterface() default java.lang.Object.class;
+ String beanName() default "";
+ String mappedName() default "";
+ String lookup() default "";
}
diff --git a/java/jakarta/el/BeanELResolver.java b/java/jakarta/el/BeanELResolver.java
index 7525e06..1226873 100644
--- a/java/jakarta/el/BeanELResolver.java
+++ b/java/jakarta/el/BeanELResolver.java
@@ -37,8 +37,7 @@ import java.util.concurrent.ConcurrentHashMap;
public class BeanELResolver extends ELResolver {
private static final int CACHE_SIZE;
- private static final String CACHE_SIZE_PROP =
- "org.apache.el.BeanELResolver.CACHE_SIZE";
+ private static final String CACHE_SIZE_PROP = "org.apache.el.BeanELResolver.CACHE_SIZE";
static {
String cacheSizeStr;
@@ -53,8 +52,7 @@ public class BeanELResolver extends ELResolver {
private final boolean readOnly;
- private final ConcurrentCache<String, BeanProperties> cache =
- new ConcurrentCache<>(CACHE_SIZE);
+ private final ConcurrentCache<String, BeanProperties> cache = new ConcurrentCache<>(CACHE_SIZE);
public BeanELResolver() {
this.readOnly = false;
diff --git a/java/jakarta/el/ExpressionFactory.java b/java/jakarta/el/ExpressionFactory.java
index 67b3556..649b77c 100644
--- a/java/jakarta/el/ExpressionFactory.java
+++ b/java/jakarta/el/ExpressionFactory.java
@@ -42,8 +42,7 @@ import java.util.concurrent.locks.ReentrantReadWriteLock;
*/
public abstract class ExpressionFactory {
- private static final boolean IS_SECURITY_ENABLED =
- (System.getSecurityManager() != null);
+ private static final boolean IS_SECURITY_ENABLED = (System.getSecurityManager() != null);
private static final String PROPERTY_NAME = "jakarta.el.ExpressionFactory";
diff --git a/java/jakarta/el/ListELResolver.java b/java/jakarta/el/ListELResolver.java
index 03c03d4..3b0bc5e 100644
--- a/java/jakarta/el/ListELResolver.java
+++ b/java/jakarta/el/ListELResolver.java
@@ -28,8 +28,7 @@ public class ListELResolver extends ELResolver {
private final boolean readOnly;
- private static final Class<?> UNMODIFIABLE =
- Collections.unmodifiableList(new ArrayList<>()).getClass();
+ private static final Class<?> UNMODIFIABLE = Collections.unmodifiableList(new ArrayList<>()).getClass();
public ListELResolver() {
this.readOnly = false;
diff --git a/java/jakarta/el/Util.java b/java/jakarta/el/Util.java
index 3051a7b..66a1ca5 100644
--- a/java/jakarta/el/Util.java
+++ b/java/jakarta/el/Util.java
@@ -363,7 +363,7 @@ class Util {
throw new MethodNotFoundException(message(
null, "util.method.ambiguous", clazz, name,
paramString(paramTypes)));
- }
+ }
}
// Handle case where no match at all was found
@@ -422,12 +422,11 @@ class Util {
}
for (Wrapper<T> c : candidates) {
- if (c.getParameterTypes()[nonMatchIndex] ==
- paramTypes[nonMatchIndex]) {
- // Methods have different non-matching parameters
- // Result is ambiguous
- return null;
- }
+ if (c.getParameterTypes()[nonMatchIndex] == paramTypes[nonMatchIndex]) {
+ // Methods have different non-matching parameters
+ // Result is ambiguous
+ return null;
+ }
}
// Can't be null
@@ -630,11 +629,8 @@ class Util {
parameterTypes[i]);
}
// Last parameter is the varargs
- Class<?> varArgClass =
- parameterTypes[varArgIndex].getComponentType();
- final Object varargs = Array.newInstance(
- varArgClass,
- (paramCount - varArgIndex));
+ Class<?> varArgClass = parameterTypes[varArgIndex].getComponentType();
+ final Object varargs = Array.newInstance(varArgClass, (paramCount - varArgIndex));
for (int i = (varArgIndex); i < paramCount; i++) {
Array.set(varargs, i - varArgIndex,
factory.coerceToType(params[i], varArgClass));
diff --git a/java/jakarta/persistence/PersistenceContext.java b/java/jakarta/persistence/PersistenceContext.java
index f53b6b2..9b5f8ca 100644
--- a/java/jakarta/persistence/PersistenceContext.java
+++ b/java/jakarta/persistence/PersistenceContext.java
@@ -25,9 +25,9 @@ import java.lang.annotation.Target;
@Retention(RetentionPolicy.RUNTIME)
public @interface PersistenceContext {
- String name() default "";
- String unitName() default "";
- PersistenceContextType type() default PersistenceContextType.TRANSACTION;
- PersistenceProperty[] properties() default {};
- SynchronizationType synchronization() default SynchronizationType.SYNCHRONIZED;
+ String name() default "";
+ String unitName() default "";
+ PersistenceContextType type() default PersistenceContextType.TRANSACTION;
+ PersistenceProperty[] properties() default {};
+ SynchronizationType synchronization() default SynchronizationType.SYNCHRONIZED;
}
diff --git a/java/jakarta/persistence/PersistenceProperty.java b/java/jakarta/persistence/PersistenceProperty.java
index 46943a5..70c51e7 100644
--- a/java/jakarta/persistence/PersistenceProperty.java
+++ b/java/jakarta/persistence/PersistenceProperty.java
@@ -25,6 +25,6 @@ import java.lang.annotation.Target;
@Retention(RetentionPolicy.RUNTIME)
public @interface PersistenceProperty {
- String name();
- String value();
+ String name();
+ String value();
}
diff --git a/java/jakarta/persistence/PersistenceUnit.java b/java/jakarta/persistence/PersistenceUnit.java
index cdbee56..bd53eaf 100644
--- a/java/jakarta/persistence/PersistenceUnit.java
+++ b/java/jakarta/persistence/PersistenceUnit.java
@@ -27,6 +27,6 @@ import java.lang.annotation.Target;
@Retention(RetentionPolicy.RUNTIME)
public @interface PersistenceUnit {
- String name() default "";
- String unitName() default "";
+ String name() default "";
+ String unitName() default "";
}
diff --git a/res/checkstyle/checkstyle.xml b/res/checkstyle/checkstyle.xml
index 88551f7..ed16bc3 100644
--- a/res/checkstyle/checkstyle.xml
+++ b/res/checkstyle/checkstyle.xml
@@ -78,7 +78,7 @@
<module name="UnusedImports"/>
<!-- Miscellaneous -->
- <!-- ~5000 errors
+ <!-- ~5500 errors
<module name="Indentation">
<property name="lineWrappingIndentation" value="8"/>
</module>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org