Merge pull request #1789 from aschackmull/java/autoformat

Java: Autoformat.

Author: yh-semmle

java/ql/src/Performance/ConcatenationInLoops.ql

@@ -57,5 +57,7 @@ predicate declaredInLoop(LocalVariableDecl v, LoopStmt loop) {
 from Assignment a, Variable v
 where
   useAndDef(a, v) and
-  exists(LoopStmt loop | a.getEnclosingStmt().getEnclosingStmt*() = loop | not declaredInLoop(v, loop))
+  exists(LoopStmt loop | a.getEnclosingStmt().getEnclosingStmt*() = loop |
+    not declaredInLoop(v, loop)
+  )
 select a, "The string " + v.getName() + " is built-up in a loop: use string buffer."

java/ql/src/semmle/code/java/Expr.qll

@@ -1117,9 +1117,7 @@ class SwitchExpr extends Expr, @switchexpr {
   Expr getAResult() {
     result = getACase().getRuleExpression()
     or
-    exists(BreakStmt break |
-      break.(JumpStmt).getTarget() = this and result = break.getValue()
-    )
+    exists(BreakStmt break | break.(JumpStmt).getTarget() = this and result = break.getValue())
   }
 
   /** Gets a printable representation of this expression. */

java/ql/src/semmle/code/java/Member.qll

@@ -120,9 +120,7 @@ class Callable extends StmtParent, Member, @callable {
    *
    * This includes both static call targets and dynamic dispatch targets.
    */
-  predicate polyCalls(Callable m) {
-    this.calls(m) or this.callsImpl(m)
-  }
+  predicate polyCalls(Callable m) { this.calls(m) or this.callsImpl(m) }
 
   /**
    * Holds if `c` is a viable implementation of a callable called by this

java/ql/src/semmle/code/java/controlflow/internal/Preconditions.qll

@@ -3,6 +3,7 @@
  * `com.google.common.base.Preconditions` and
  * `org.apache.commons.lang3.Validate`.
  */
+
 import java
 
 /**

java/ql/src/semmle/code/java/dataflow/TaintTracking.qll

@@ -2,9 +2,9 @@
  * Provides classes for performing local (intra-procedural) and
  * global (inter-procedural) taint-tracking analyses.
  */
+
 import semmle.code.java.dataflow.DataFlow
 import semmle.code.java.dataflow.DataFlow2
-
 import semmle.code.java.dataflow.internal.TaintTrackingUtil::StringBuilderVarModule
 
 module TaintTracking {
@@ -14,6 +14,5 @@ module TaintTracking {
   /**
    * DEPRECATED: Use TaintTracking2::Configuration instead.
    */
-  deprecated
-  class Configuration2 = TaintTracking2::Configuration;
+  deprecated class Configuration2 = TaintTracking2::Configuration;
 }

java/ql/src/semmle/code/java/dataflow/internal/TaintTrackingUtil.qll

@@ -252,9 +252,7 @@ private predicate taintPreservingQualifierToMethod(Method m) {
     m.getName() = "trim"
   )
   or
-  exists(Class c | c.getQualifiedName() = "java.lang.Number" |
-    hasSubtype*(c, m.getDeclaringType())
-  ) and
+  exists(Class c | c.getQualifiedName() = "java.lang.Number" | hasSubtype*(c, m.getDeclaringType())) and
   (
     m.getName().matches("to%String") or
     m.getName() = "toByteArray" or
@@ -555,8 +553,8 @@ class ObjectOutputStreamVar extends LocalVariableDecl {
     result.getMethod().hasName("writeObject")
   }
 }
-
 private import StringBuilderVarModule
+
 module StringBuilderVarModule {
   /**
    * A local variable that is initialized to a `StringBuilder`

java/ql/src/semmle/code/java/dataflow/internal/tainttracking1/TaintTrackingImpl.qll

@@ -85,4 +85,3 @@ abstract class Configuration extends DataFlow::Configuration {
     super.hasFlow(source, sink)
   }
 }
-

java/ql/src/semmle/code/java/dataflow/internal/tainttracking2/TaintTrackingImpl.qll

@@ -85,4 +85,3 @@ abstract class Configuration extends DataFlow::Configuration {
     super.hasFlow(source, sink)
   }
 }
-

java/ql/src/semmle/code/java/dispatch/ObjFlow.qll

@@ -6,6 +6,7 @@
  * The set of dispatch targets for `Object.toString()` calls are reduced based
  * on possible data flow from objects of more specific types to the qualifier.
  */
+
 import java
 private import VirtualDispatch
 private import semmle.code.java.controlflow.Guards

java/ql/test/library-tests/frameworks/android/manifest/manifest.ql

@@ -2,4 +2,5 @@ import java
 import semmle.code.xml.AndroidManifest
 
 from AndroidActivityXmlElement e
-select e.getResolvedComponentName(), e.getAnIntentFilterElement().getAnActionElement().getActionName()
+select e.getResolvedComponentName(),
+  e.getAnIntentFilterElement().getAnActionElement().getActionName()