Skip to content

Commit caae5a8

Browse files
hvitvedCopilot
andauthored
Apply suggestions from code review
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
1 parent 0937133 commit caae5a8

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

rust/ql/lib/codeql/rust/elements/internal/VariableImpl.qll

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ module Impl {
7979
* are parsed left-associatively, so the AST for the condition looks like
8080
*
8181
* ```rust
82-
* ((let x1 = ... && let x2 = ...) && ...) & let xn = ...
82+
* ((let x1 = ... && let x2 = ...) && ...) && let xn = ...
8383
* ```
8484
*
8585
* This, however, does not work with scoping and shadowing, so we instead treat
@@ -319,10 +319,10 @@ module Impl {
319319
result.(Param).getPat() = getAPatAncestor*(this.getPat())
320320
}
321321

322-
/** Hold is this variable is mutable. */
322+
/** Holds if this variable is mutable. */
323323
predicate isMutable() { this.getPat().isMut() or this.getSelfParam().isMut() }
324324

325-
/** Hold is this variable is immutable. */
325+
/** Holds if this variable is immutable. */
326326
predicate isImmutable() { not this.isMutable() }
327327
}
328328

shared/namebinding/codeql/namebinding/LocalNameBinding.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,7 +132,7 @@ signature module LocalNameBindingInputSig<LocationSig Location> {
132132
*
133133
* If `scope` declares a local with the name of `n`, then `scope` is guaranteed
134134
* to be the scope that `n` ultimately resolves to. This can thus be used to take
135-
* full control of scope resolution for for specific types of references.
135+
* full control of scope resolution for specific types of references.
136136
*/
137137
default predicate lookupStartsAt(AstNode n, AstNode scope) { none() }
138138
}

0 commit comments

Comments
 (0)