Fix recursion

[User4martin]

Fix #324 comment 1. Subsequent (side-by-side) recursive calls did see data captured in previous calls. Improve speed.

Include existing PR #323 and #322

@Alexey-T

All tests work for the old and new recursion behaviour. Except

IsMatching('2nd recurse does NOT see capture from earlier recurse',
             '[aA](?R)?(?:X|([bcBC]))(?R)?\1',
             'aABBcAXBc',  [2,3,  3,1]);

  IsMatching('2nd recurse does NOT see capture from earlier recurse',
             '[aA]((?R))?(?:X|([bcBC]))((?R))?\2',
             'aABBcAXBc',  [2,3,  -1,-1,  3,1,  -1,-1]);

Which are specially crafted to exploit the issue in the old behaviour

[Alexey-T]

@andgineer Pls apply.

[andgineer]

could you resolve the conflicts?

[User4martin]

Will do...

Can you check https://github.com/User4martin/TRegExpr/commit/more-references Can that be added? https://regex101.com/r/7qgquy/1

(and one other fix coming up)

[Alexey-T]

Can that be added? https://regex101.com/r/7qgquy/1

It must be discussed in a new topic. with detailed description: what was not ok, what you propose...

[User4martin]

Added #326 Added fix for "literal {" mode

[Alexey-T]

Write to history block ending: "by Martin Friebe." Better increase version to 1.167.

[User4martin]

Added another fix. Checking fixed length of a look-behind would fail if the look-behind had certain reference or sub-calls. (Because the HasFixedLength would increment the pointer to the next OP, but not skip the index stored after the command.)

[Alexey-T]

docs:

``\k`name```

Did you make mistake - backtick instead of single quote?

[User4martin]

@Alexey-T

• Added: "GrpEnd" is guarded by "GrpStart<>nil" => no need to "FillChar" (speed up)
• fix: An earlier (already merged) commit added "default param"
• GrpSubCalled is only needed in execution loop, and therefore should only be cleared for a new Exec call

[Alexey-T]

Good, tkx.

 (GrpBounds[0].GrpEnd[Idx] > GrpBounds[0].GrpStart[Idx])

maybe better >= ?

[User4martin]

Good, tkx.

 (GrpBounds[0].GrpEnd[Idx] > GrpBounds[0].GrpStart[Idx])

maybe better >= ?

That is in TRegExpr.GetMatch ?

If it is equal, then the result is an empty string. And it has already been set to empty.

[Alexey-T]

Ok, indeed.

[Alexey-T]

@andgineer Please apply?