Skip to content

Commit 5374506

Browse files
committed
use a consistent comment about the ignore case flag
1 parent ff74fe1 commit 5374506

File tree

1 file changed

+2
-2
lines changed
  • javascript/ql/lib/semmle/javascript/security/performance

1 file changed

+2
-2
lines changed

javascript/ql/lib/semmle/javascript/security/performance/ReDoSUtil.qll

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,7 @@ private newtype TInputSymbol =
201201
not RegExpFlags::isIgnoreCase(cc.getRootTerm())
202202
).getValue().charAt(_)
203203
or
204-
// normalize to lower case if the regexp is case insensitive
204+
// normalize everything to lower case if the regexp is case insensitive
205205
c =
206206
any(RegexpCharacterConstant cc, string char |
207207
cc instanceof RelevantRegExpTerm and
@@ -577,7 +577,7 @@ predicate delta(State q1, EdgeLabel lbl, State q2) {
577577
not RegExpFlags::isIgnoreCase(s.getRootTerm()) and
578578
lbl = Char(s.getValue().charAt(i))
579579
or
580-
// normalizing to lower case if ignorecase flag is set
580+
// normalize everything to lower case if the regexp is case insensitive
581581
RegExpFlags::isIgnoreCase(s.getRootTerm()) and
582582
exists(string c | c = s.getValue().charAt(i) | lbl = Char(c.toLowerCase()))
583583
) and

0 commit comments

Comments
 (0)