For user class ']'at the beginning does not included into class about tregexpr HOT 9 CLOSED

@Alexey-T could you fix that? and add tests for re like '[[ab]', '[]ab]'?

from tregexpr.

Will try it

from tregexpr.

Line 1010 is in the func RegExprSubExpressions, so the issue is in this func? Or it’s everywhere? Repro steps?

from tregexpr.

It checks for “]” as first char inside [] and in this case includes it into the char set.
But also it have to check also for “[” so we can use re “[[ab]*” to find string like “a[b”

To chech that we need tests with re like “[[ab]” and “[]ab]”

from tregexpr.

May be I found wrong line - it shoud be inside re parser

from tregexpr.

What wrong line, sorry ?

from tregexpr.

Line 1010
As I see it should be

but we have to support this syntax in RegExprSubExpressions too

from tregexpr.

Pull request, with test 38

from tregexpr.

silly me - we do not need to check for '[' because user class ("[]") includes just any char but ']'.
so we should check only for ']' at first position.

from tregexpr.