Skip to content

Commit e0c331d

Browse files
committed
Swift: Make implicit this receivers explicit
1 parent bcdd839 commit e0c331d

File tree

11 files changed

+20
-20
lines changed

11 files changed

+20
-20
lines changed

swift/ql/lib/codeql/swift/elements/AvailabilityInfo.qll

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@ private import codeql.swift.generated.AvailabilityInfo
22

33
class AvailabilityInfo extends Generated::AvailabilityInfo {
44
override string toString() {
5-
result = "#available" and not isUnavailable()
5+
result = "#available" and not this.isUnavailable()
66
or
7-
result = "#unavailable" and isUnavailable()
7+
result = "#unavailable" and this.isUnavailable()
88
}
99
}

swift/ql/lib/codeql/swift/elements/KeyPathComponent.qll

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,37 +5,37 @@ class KeyPathComponent extends Generated::KeyPathComponent {
55
/**
66
* Property access like `.bar` in `\Foo.bar`.
77
*/
8-
predicate isProperty() { getKind() = 3 }
8+
predicate isProperty() { this.getKind() = 3 }
99

1010
/**
1111
* Array or dictionary subscript like `[1]` or `["a", "b"]`.
1212
*/
13-
predicate isSubscript() { getKind() = 4 }
13+
predicate isSubscript() { this.getKind() = 4 }
1414

1515
/**
1616
* Optional forcing `!`.
1717
*/
18-
predicate isOptionalForcing() { getKind() = 5 }
18+
predicate isOptionalForcing() { this.getKind() = 5 }
1919

2020
/**
2121
* Optional chaining `?`.
2222
*/
23-
predicate isOptionalChaining() { getKind() = 6 }
23+
predicate isOptionalChaining() { this.getKind() = 6 }
2424

2525
/**
2626
* Implicit optional wrapping component inserted by the compiler when an optional chain ends in a non-optional value.
2727
*/
28-
predicate isOptionalWrapping() { getKind() = 7 }
28+
predicate isOptionalWrapping() { this.getKind() = 7 }
2929

3030
/**
3131
* Reference to the entire object; the `self` in `\Foo.self`.
3232
*/
33-
predicate isSelf() { getKind() = 8 }
33+
predicate isSelf() { this.getKind() = 8 }
3434

3535
/**
3636
* Tuple indexing like `.1`.
3737
*/
38-
predicate isTupleIndexing() { getKind() = 9 }
38+
predicate isTupleIndexing() { this.getKind() = 9 }
3939

4040
/** Gets the underlying key-path expression which this is a component of. */
4141
KeyPathExpr getKeyPathExpr() { result.getAComponent() = this }

swift/ql/lib/codeql/swift/elements/Locatable.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,5 +14,5 @@ class Locatable extends Generated::Locatable {
1414
/**
1515
* Gets the primary file where this element occurs.
1616
*/
17-
File getFile() { result = getLocation().getFile() }
17+
File getFile() { result = this.getLocation().getFile() }
1818
}
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
private import codeql.swift.generated.PlatformVersionAvailabilitySpec
22

33
class PlatformVersionAvailabilitySpec extends Generated::PlatformVersionAvailabilitySpec {
4-
override string toString() { result = getPlatform() + " " + getVersion() }
4+
override string toString() { result = this.getPlatform() + " " + this.getVersion() }
55
}

swift/ql/lib/codeql/swift/elements/decl/ExtensionDecl.qll

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,9 @@ private import codeql.swift.generated.decl.ExtensionDecl
22

33
class ExtensionDecl extends Generated::ExtensionDecl {
44
override string toString() {
5-
result = "extension of " + getExtendedTypeDecl().toString()
5+
result = "extension of " + this.getExtendedTypeDecl().toString()
66
or
7-
not exists(getExtendedTypeDecl()) and
7+
not exists(this.getExtendedTypeDecl()) and
88
result = "extension"
99
}
1010
}
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
private import codeql.swift.generated.expr.IdentityExpr
22

33
class IdentityExpr extends Generated::IdentityExpr {
4-
override predicate convertsFrom(Expr e) { e = getImmediateSubExpr() }
4+
override predicate convertsFrom(Expr e) { e = this.getImmediateSubExpr() }
55
}

swift/ql/lib/codeql/swift/elements/expr/UnresolvedDeclRefExpr.qll

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@ private import codeql.swift.generated.expr.UnresolvedDeclRefExpr
22

33
class UnresolvedDeclRefExpr extends Generated::UnresolvedDeclRefExpr {
44
override string toString() {
5-
result = getName() + " (unresolved)"
5+
result = this.getName() + " (unresolved)"
66
or
7-
not hasName() and result = "(unresolved)"
7+
not this.hasName() and result = "(unresolved)"
88
}
99
}
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
private import codeql.swift.generated.expr.UnresolvedDotExpr
22

33
class UnresolvedDotExpr extends Generated::UnresolvedDotExpr {
4-
override string toString() { result = "... ." + getName() }
4+
override string toString() { result = "... ." + this.getName() }
55
}
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
private import codeql.swift.generated.pattern.BindingPattern
22

33
class BindingPattern extends Generated::BindingPattern {
4-
final override Pattern getResolveStep() { result = getImmediateSubPattern() }
4+
final override Pattern getResolveStep() { result = this.getImmediateSubPattern() }
55

66
override string toString() { result = "let ..." }
77
}
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
private import codeql.swift.generated.pattern.ParenPattern
22

33
class ParenPattern extends Generated::ParenPattern {
4-
final override Pattern getResolveStep() { result = getImmediateSubPattern() }
4+
final override Pattern getResolveStep() { result = this.getImmediateSubPattern() }
55

66
override string toString() { result = "(...)" }
77
}

0 commit comments

Comments
 (0)