[colorer/Colorer-library] cregexp_refactoring: Drop Perl-style /m tests; Colorer regexps are line-based. (98f807f)

0 views
Skip to first unread message

farg...@farmanager.com

unread,
Aug 16, 2026, 1:46:15 PM (14 hours ago) Aug 16
to farco...@googlegroups.com
Repository : https://github.com/colorer/Colorer-library
On branch : cregexp_refactoring
Link : https://github.com/colorer/Colorer-library/commit/98f807f91be6cf5285d3fbcacfedc5aefbac04a0

>---------------------------------------------------------------

commit 98f807f91be6cf5285d3fbcacfedc5aefbac04a0
Author: Aleksey Dobrunov <cta...@ctapmex.com>
Date: Sun Aug 16 21:24:08 2026 +0500

Drop Perl-style /m tests; Colorer regexps are line-based.


>---------------------------------------------------------------

98f807f91be6cf5285d3fbcacfedc5aefbac04a0
tests/unit/test_cregexp.cpp | 12 ------------
1 file changed, 12 deletions(-)

diff --git a/tests/unit/test_cregexp.cpp b/tests/unit/test_cregexp.cpp
index f98c010..8b7af25 100644
--- a/tests/unit/test_cregexp.cpp
+++ b/tests/unit/test_cregexp.cpp
@@ -235,12 +235,6 @@ TEST_CASE("CRegExp flags", "[cregexp]")
{
require_match(u"/a b c/x", u"abc", 0, 3);
}
-
- SECTION("caret matches after newline with /m")
- {
- require_match(u"/^cd/m", u"ab\ncd", 3, 5, true);
- require_no_match(u"/^cd/", u"ab\ncd", true);
- }
}

TEST_CASE("CRegExp greedy and lazy quantifiers", "[cregexp]")
@@ -555,9 +549,3 @@ TEST_CASE("CRegExp \\Y{name} copies named group case-insensitively", "[cregexp]"
REQUIRE(end_match.s[0] == 0);
REQUIRE(end_match.e[0] == 3);
}
-
-TEST_CASE("CRegExp $ matches before newline with /m", "[cregexp][bugs]")
-{
- SKIP("$ with /m currently checks the previous character, not the current one");
- require_match(u"/\\w+$/m", u"ab\ncd", 0, 2);
-}


Reply all
Reply to author
Forward
0 new messages