Enforce errorprone UnnecessaryParentheses (#274)

trinodb · Mar 5, 2024 · 68be752 · 68be752
1 parent 43c8142
commit 68be752
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/gateway-ha/src/main/java/io/trino/gateway/baseapp/BaseApp.java b/gateway-ha/src/main/java/io/trino/gateway/baseapp/BaseApp.java
@@ -258,7 +258,7 @@ private void registerAuthFilters(Environment environment, Injector injector)
     {
         environment
                 .jersey()
-                .register(new AuthDynamicFeature((injector.getInstance(AuthFilter.class))));
+                .register(new AuthDynamicFeature(injector.getInstance(AuthFilter.class)));
         logger.info("op=register type=auth filter item={}", AuthFilter.class);
         environment.jersey().register(RolesAllowedDynamicFeature.class);
     }

diff --git a/gateway-ha/src/main/java/io/trino/gateway/proxyserver/ProxyHandler.java b/gateway-ha/src/main/java/io/trino/gateway/proxyserver/ProxyHandler.java
@@ -112,7 +112,7 @@ protected String plainTextFromGz(byte[] compressed)
 
     protected boolean isCompressed(final byte[] compressed)
     {
-        return (compressed[0] == (byte) (GZIPInputStream.GZIP_MAGIC))
+        return (compressed[0] == (byte) GZIPInputStream.GZIP_MAGIC)
                 && (compressed[1] == (byte) (GZIPInputStream.GZIP_MAGIC >> 8));
     }
 }
diff --git a/pom.xml b/pom.xml
@@ -97,6 +97,7 @@
                     <compilerArgs>
                         <arg>-XDcompilePolicy=simple</arg>
                         <arg>-Xplugin:ErrorProne \
+                            -Xep:UnnecessaryParentheses:ERROR \
                             -Xep:UnusedVariable:ERROR</arg>
                     </compilerArgs>
                     <annotationProcessorPaths>