hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2026-07-31 07:22:56 +02:00
Code Issues Packages Projects Releases Wiki Activity

C++: Remove Ruby-only \h and \H character class escapes

Browse Source
This commit is contained in:
copilot-swe-agent[bot]
2026-07-23 10:06:02 +00:00
committed by GitHub
parent f75d66dc91
commit 72bc7170ff
4 changed files with 1 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
cpp/ql/lib/semmle/code/cpp/regex/RegexTreeView.qll
View File

@@ -651,7 +651,7 @@ private module Impl implements RegexTreeViewSig {
* ```
*/
class RegExpCharacterClassEscape extends RegExpEscape {
RegExpCharacterClassEscape() { this.getValue() in ["d", "D", "s", "S", "w", "W", "h", "H"] }
RegExpCharacterClassEscape() { this.getValue() in ["d", "D", "s", "S", "w", "W"] }
override RegExpTerm getChild(int i) { none() }

8
cpp/ql/test/library-tests/regex/parse.expected
View File

@@ -86,10 +86,6 @@
#-----| [RegExpConstant, RegExpEscape] \t
#-----| [RegExpCharacterClassEscape] \h
#-----| [RegExpCharacterClassEscape] \H
#-----| [RegExpCharacterClassEscape] \d
#-----| [RegExpCharacterClassEscape] \D
@@ -226,10 +222,6 @@
#-----| 1 -> [RegExpConstant, RegExpEscape] \r
#-----| 2 -> [RegExpConstant, RegExpEscape] \t
#-----| [RegExpSequence] \h\H
#-----| 0 -> [RegExpCharacterClassEscape] \h
#-----| 1 -> [RegExpCharacterClassEscape] \H
#-----| [RegExpSequence] \d\D
#-----| 0 -> [RegExpCharacterClassEscape] \d
#-----| 1 -> [RegExpCharacterClassEscape] \D

1
cpp/ql/test/library-tests/regex/regexp.cpp
View File

@@ -47,7 +47,6 @@ std::regex r_meta1m(".*"); // /.*/m mode variant — mode flags are constructor
std::regex r_meta2("\\w+\\W");
std::regex r_meta3("\\s\\S");
std::regex r_meta4("\\d\\D");
std::regex r_meta5("\\h\\H");
std::regex r_meta6("\\n\\r\\t");
// Anchors

5
cpp/ql/test/library-tests/regex/regexp.expected
View File

@@ -76,7 +76,6 @@ term
| file://:0:0:0:0 | \\1 | RegExpBackRef |
| file://:0:0:0:0 | \\B | RegExpNonWordBoundary |
| file://:0:0:0:0 | \\D | RegExpCharacterClassEscape |
| file://:0:0:0:0 | \\H | RegExpCharacterClassEscape |
| file://:0:0:0:0 | \\P{Digit} | RegExpNamedCharacterProperty |
| file://:0:0:0:0 | \\P{Digit}+ | RegExpPlus |
| file://:0:0:0:0 | \\S | RegExpCharacterClassEscape |
@@ -88,8 +87,6 @@ term
| file://:0:0:0:0 | \\b!a\\B | RegExpSequence |
| file://:0:0:0:0 | \\d | RegExpCharacterClassEscape |
| file://:0:0:0:0 | \\d\\D | RegExpSequence |
| file://:0:0:0:0 | \\h | RegExpCharacterClassEscape |
| file://:0:0:0:0 | \\h\\H | RegExpSequence |
| file://:0:0:0:0 | \\k<qux> | RegExpBackRef |
| file://:0:0:0:0 | \\k<qux>+ | RegExpPlus |
| file://:0:0:0:0 | \\n | RegExpConstant,RegExpEscape |
@@ -208,13 +205,11 @@ regExpNormalCharValue
| file://:0:0:0:0 | Z | Z |
| file://:0:0:0:0 | [ | [ |
| file://:0:0:0:0 | \\D | D |
| file://:0:0:0:0 | \\H | H |
| file://:0:0:0:0 | \\S | S |
| file://:0:0:0:0 | \\W | W |
| file://:0:0:0:0 | \\[ | [ |
| file://:0:0:0:0 | \\] | ] |
| file://:0:0:0:0 | \\d | d |
| file://:0:0:0:0 | \\h | h |
| file://:0:0:0:0 | \\n | \n |
| file://:0:0:0:0 | \\n | \n |
| file://:0:0:0:0 | \\r | \r |