Update Logging.qll

bananabr · web-flow · commit 032a7e71fe63 · 2021-09-16T13:03:26.000-05:00
Simplified using a set-literal as suggested by @intrigus-lgtm
diff --git a/java/ql/src/experimental/semmle/code/java/Logging.qll b/java/ql/src/experimental/semmle/code/java/Logging.qll
@@ -30,12 +30,7 @@ class LoggingCall extends MethodAccess {
     or
     exists(RefType t, Method m | t.hasQualifiedName("android.util", "Log") |
       (
-        m.hasName("d") or
-        m.hasName("e") or
-        m.hasName("i") or
-        m.hasName("v") or
-        m.hasName("w") or
-        m.hasName("wtf")
+        m.hasName(["d", "e", "i", "v", "w", "wtf"])
       ) and
       (
         m.getDeclaringType().getASourceSupertype*() = t or

Original file line number	Diff line number	Diff line change
`@@ -30,12 +30,7 @@ class LoggingCall extends MethodAccess {`
`30`	`30`	`or`
`31`	`31`	`exists(RefType t, Method m \| t.hasQualifiedName("android.util", "Log") \|`
`32`	`32`	`(`
`33`		`- m.hasName("d") or`
`34`		`- m.hasName("e") or`
`35`		`- m.hasName("i") or`
`36`		`- m.hasName("v") or`
`37`		`- m.hasName("w") or`
`38`		`- m.hasName("wtf")`
	`33`	`+ m.hasName(["d", "e", "i", "v", "w", "wtf"])`
`39`	`34`	`) and`
`40`	`35`	`(`
`41`	`36`	`m.getDeclaringType().getASourceSupertype*() = t or`