Allow closing paren to be on next line

Allow (but don't require) the closing parenthesis of a method invocation to be on the following line. Signed-off-by: Andrew Rouse <[email protected]>
jakartaee · Dec 13, 2023 · 104753e · 104753e
1 parent ff15f60
commit 104753e
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 5 deletions.
diff --git a/api/src/main/java/jakarta/enterprise/inject/build/compatible/spi/SecurityActions.java b/api/src/main/java/jakarta/enterprise/inject/build/compatible/spi/SecurityActions.java
@@ -26,6 +26,7 @@ private SecurityActions() {
 
     static <T> ServiceLoader<T> loadService(Class<T> service, ClassLoader classLoader) {
         return AccessController.doPrivileged(
-                (PrivilegedAction<ServiceLoader<T>>) () -> ServiceLoader.load(service, classLoader));
+                (PrivilegedAction<ServiceLoader<T>>) () -> ServiceLoader.load(service, classLoader)
+        );
     }
 }
diff --git a/api/src/main/java/jakarta/enterprise/inject/spi/SecurityActions.java b/api/src/main/java/jakarta/enterprise/inject/spi/SecurityActions.java
@@ -35,6 +35,7 @@ private SecurityActions() {
 
     static <T> ServiceLoader<T> loadService(Class<T> service, ClassLoader classLoader) {
         return AccessController.doPrivileged(
-                (PrivilegedAction<ServiceLoader<T>>) () -> ServiceLoader.load(service, classLoader));
+                (PrivilegedAction<ServiceLoader<T>>) () -> ServiceLoader.load(service, classLoader)
+        );
     }
 }
diff --git a/api/src/main/java/jakarta/enterprise/util/SecurityActions.java b/api/src/main/java/jakarta/enterprise/util/SecurityActions.java
@@ -39,7 +39,8 @@ static void setAccessible(Method method) {
                     (PrivilegedAction<?>) () -> {
                         method.setAccessible(true);
                         return null;
-                    });
+                    }
+            );
         } else {
             method.setAccessible(true);
         }
@@ -48,7 +49,8 @@ static void setAccessible(Method method) {
     static Method[] getDeclaredMethods(Class<?> clazz) {
         if (System.getSecurityManager() != null) {
             return AccessController.doPrivileged(
-                    (PrivilegedAction<Method[]>) () -> clazz.getDeclaredMethods());
+                    (PrivilegedAction<Method[]>) () -> clazz.getDeclaredMethods()
+            );
         } else {
             return clazz.getDeclaredMethods();
         }

diff --git a/ide-config/src/main/resources/cdi-format.xml b/ide-config/src/main/resources/cdi-format.xml
@@ -305,7 +305,7 @@
 <setting id="org.eclipse.jdt.core.formatter.parentheses_positions_in_if_while_statement" value="common_lines"/>
 <setting id="org.eclipse.jdt.core.formatter.parentheses_positions_in_lambda_declaration" value="common_lines"/>
 <setting id="org.eclipse.jdt.core.formatter.parentheses_positions_in_method_delcaration" value="common_lines"/>
-<setting id="org.eclipse.jdt.core.formatter.parentheses_positions_in_method_invocation" value="common_lines"/>
+<setting id="org.eclipse.jdt.core.formatter.parentheses_positions_in_method_invocation" value="preserve_positions"/>
 <setting id="org.eclipse.jdt.core.formatter.parentheses_positions_in_switch_statement" value="common_lines"/>
 <setting id="org.eclipse.jdt.core.formatter.parentheses_positions_in_try_clause" value="common_lines"/>
 <setting id="org.eclipse.jdt.core.formatter.put_empty_statement_on_new_line" value="true"/>