Skip to content

Commit f646938

Browse files
committed
Revert "Kotlin: Adjust ExtensionReceiverAccess.toString to not return hard coded this value"
This reverts commit 9649a8f.
1 parent ac5219f commit f646938

File tree

3 files changed

+9
-11
lines changed

3 files changed

+9
-11
lines changed

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

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1894,18 +1894,16 @@ class VarAccess extends Expr, @varaccess {
18941894
* An access to an extension receiver parameter. This is a parameter access that takes the form of `this` in Kotlin.
18951895
*/
18961896
class ExtensionReceiverAccess extends VarAccess {
1897-
Parameter p;
1898-
18991897
ExtensionReceiverAccess() {
1900-
this.getVariable() = p and
1901-
p.isExtensionParameter()
1898+
exists(Parameter p |
1899+
this.getVariable() = p and
1900+
p.isExtensionParameter()
1901+
)
19021902
}
19031903

19041904
override string getAPrimaryQlClass() { result = "ExtensionReceiverAccess" }
19051905

1906-
override string toString() {
1907-
if p.getName() = "<this>" then result = "this" else result = p.getName()
1908-
}
1906+
override string toString() { result = "this" }
19091907
}
19101908

19111909
/**

java/ql/test/kotlin/library-tests/methods/exprs.expected

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -306,10 +306,10 @@
306306
| methods3.kt:3:1:3:49 | fooBarTopLevelMethodExt(...) | MethodAccess |
307307
| methods3.kt:3:1:3:49 | int | TypeAccess |
308308
| methods3.kt:3:1:3:49 | int | TypeAccess |
309-
| methods3.kt:3:1:3:49 | p0 | ExtensionReceiverAccess |
310309
| methods3.kt:3:1:3:49 | p1 | VarAccess |
311310
| methods3.kt:3:1:3:49 | p1 | VarAccess |
312311
| methods3.kt:3:1:3:49 | p2 | VarAccess |
312+
| methods3.kt:3:1:3:49 | this | ExtensionReceiverAccess |
313313
| methods3.kt:3:5:3:10 | String | TypeAccess |
314314
| methods3.kt:3:36:3:45 | int | TypeAccess |
315315
| methods3.kt:3:45:3:45 | 1 | IntegerLiteral |
@@ -327,10 +327,10 @@
327327
| methods3.kt:6:5:6:45 | int | TypeAccess |
328328
| methods3.kt:6:5:6:45 | int | TypeAccess |
329329
| methods3.kt:6:5:6:45 | p0 | VarAccess |
330-
| methods3.kt:6:5:6:45 | p1 | ExtensionReceiverAccess |
331330
| methods3.kt:6:5:6:45 | p2 | VarAccess |
332331
| methods3.kt:6:5:6:45 | p2 | VarAccess |
333332
| methods3.kt:6:5:6:45 | p3 | VarAccess |
333+
| methods3.kt:6:5:6:45 | this | ExtensionReceiverAccess |
334334
| methods3.kt:6:9:6:14 | String | TypeAccess |
335335
| methods3.kt:6:32:6:41 | int | TypeAccess |
336336
| methods3.kt:6:41:6:41 | 1 | IntegerLiteral |

java/ql/test/kotlin/library-tests/parameter-defaults/PrintAst.expected

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -566,7 +566,7 @@ test.kt:
566566
# 19| 2: [ReturnStmt] return ...
567567
# 19| 0: [MethodAccess] f(...)
568568
# 19| -1: [VarAccess] p0
569-
# 19| 0: [ExtensionReceiverAccess] p1
569+
# 19| 0: [ExtensionReceiverAccess] this
570570
# 19| 1: [VarAccess] p2
571571
# 19| 2: [VarAccess] p3
572572
# 19| 3: [VarAccess] p4
@@ -774,7 +774,7 @@ test.kt:
774774
# 56| 2: [ReturnStmt] return ...
775775
# 56| 0: [MethodAccess] test(...)
776776
# 56| -1: [VarAccess] p0
777-
# 56| 0: [ExtensionReceiverAccess] p1
777+
# 56| 0: [ExtensionReceiverAccess] this
778778
# 56| 1: [VarAccess] p2
779779
# 56| 2: [VarAccess] p3
780780
# 56| 3: [VarAccess] p4

0 commit comments

Comments
 (0)