Exclude .kt files from useless parameter query

tamasvajk · tamasvajk · commit 48c37a2c0f22 · 2022-11-16T14:51:49.000+01:00
diff --git a/java/ql/lib/semmle/code/java/deadcode/DeadCode.qll b/java/ql/lib/semmle/code/java/deadcode/DeadCode.qll
@@ -274,7 +274,7 @@ class DeadMethod extends Callable {
 
 class RootdefCallable extends Callable {
   RootdefCallable() {
-    this.fromSource() and
+    this.getFile().isJavaSourceFile() and
     not this.(Method).overridesOrInstantiates(_)
   }
 
diff --git a/java/ql/test/kotlin/query-tests/UselessParameter/UselessParameter.expected b/java/ql/test/kotlin/query-tests/UselessParameter/UselessParameter.expected
@@ -1,3 +0,0 @@
-| Test.kt:11:8:11:18 | a | The parameter 'a' is never used. |
-| Test.kt:19:5:19:5 | <this> | The parameter '<this>' is never used. |
-| Test.kt:24:9:24:14 | a | The parameter 'a' is never used. |

Original file line number	Diff line number	Diff line change
`@@ -274,7 +274,7 @@ class DeadMethod extends Callable {`
`274`	`274`
`275`	`275`	`class RootdefCallable extends Callable {`
`276`	`276`	`RootdefCallable() {`
`277`		`- this.fromSource() and`
	`277`	`+ this.getFile().isJavaSourceFile() and`
`278`	`278`	`not this.(Method).overridesOrInstantiates(_)`
`279`	`279`	`}`
`280`	`280`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +0,0 @@`
`1`		`-\| Test.kt:11:8:11:18 \| a \| The parameter 'a' is never used. \|`
`2`		`-\| Test.kt:19:5:19:5 \| <this> \| The parameter '<this>' is never used. \|`
`3`		`-\| Test.kt:24:9:24:14 \| a \| The parameter 'a' is never used. \|`