hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2025-12-21 11:16:30 +01:00
Code Issues Packages Projects Releases Wiki Activity

Apply suggestions from code review

Browse Source 
Co-Authored-By: Max Schaefer <54907921+max-schaefer@users.noreply.github.com>
This commit is contained in:
Asger F
2020-01-06 14:44:02 +00:00
committed by GitHub
parent 4c25d84b6e
commit b604be5cfb
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
javascript/ql/src/RegExp/RegExpAlwaysMatches.qhelp
View File

@@ -32,7 +32,7 @@ regular expression accordingly, such as by changing a <code>*</code> to a <code>
</recommendation> </recommendation>
<example> <example>
<p> <p>
In the following example, a regular expression is used to check the format of an string <code>id</code>. In the following example, a regular expression is used to check the format of a string <code>id</code>.
However, the check always passes because the regular expression can match the empty substring. However, the check always passes because the regular expression can match the empty substring.
For example, it will allow the ID string "<code>%%</code>" by matching an empty string at index 0. For example, it will allow the ID string "<code>%%</code>" by matching an empty string at index 0.
</p> </p>

2
javascript/ql/src/RegExp/RegExpAlwaysMatches.ql
View File

@@ -1,6 +1,6 @@
/** /**
* @name Regular expression always matches * @name Regular expression always matches
* @description Regular expression tests checks always find a match indicate dead code or a logic error * @description Regular expression tests that always find a match indicate dead code or a logic error
* @kind problem * @kind problem
* @problem.severity warning * @problem.severity warning
* @id js/regex/always-matches * @id js/regex/always-matches