mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-07-31 13:19:05 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

LibRegex: Take trailing inversion state into account in block comparison

Browse source 
Fixes #3421.
This commit is contained in:
Ali Mohammad Pur 2025-02-01 07:59:29 +01:00 • committed by Jelle Raaijmakers
commit 08ebfaff17
Notes: github-actions[bot] 2025-02-01 10:31:02 +00:00
2 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
Libraries/LibRegex/RegexOptimizer.cpp
View file

@ -500,7 +500,8 @@ static bool has_overlap(Vector<CompareTypeAndValuePair> const& lhs, Vector<Compa
} }
} }
return false; // We got to the end, just double-check that the inverse flag was not left on (which would match everything).
return current_lhs_inversion_state();
} }
enum class AtomicRewritePreconditionResult { enum class AtomicRewritePreconditionResult {

1
Tests/LibRegex/Regex.cpp
View file

@ -715,6 +715,7 @@ TEST_CASE(ECMA262_match)
{ "^(\\d{4}|[+-]\\d{6})(?:-?(\\d{2})(?:-?(\\d{2}))?)?(?:[ T]?(\\d{2}):?(\\d{2})(?::?(\\d{2})(?:[,.](\\d{1,}))?)?(?:(Z)|([+-])(\\d{2})(?::?(\\d{2}))?)?)?$"sv, { "^(\\d{4}|[+-]\\d{6})(?:-?(\\d{2})(?:-?(\\d{2}))?)?(?:[ T]?(\\d{2}):?(\\d{2})(?::?(\\d{2})(?:[,.](\\d{1,}))?)?(?:(Z)|([+-])(\\d{2})(?::?(\\d{2}))?)?)?$"sv,
""sv, ""sv,
false, }, // See above, also ladybird#2931. false, }, // See above, also ladybird#2931.
{ "[^]*[^]"sv, "i"sv, true }, // Optimizer bug, ignoring an enabled trailing 'invert' when comparing blocks, ladybird#3421.
}; };
// clang-format on // clang-format on