unnecessary PREG_OFFSET_CAPTURE usage (?)

[ViliusS]

Just wondering is PREG_OFFSET_CAPTURE is needed here https://github.com/thunderer/Shortcode/blob/65d8b2937430e944c84a0123d2b0dd392c1c85c7/src/Parser/RegularParser.php#L338 I don't see offset used anywhere in the following code however during tests I found that it uses up memory pretty heavily. If I did my testing right, in one case preg_match_all() uses up 80 MB with PREG_OFFSET_CAPTURE and 23 MB without.

[thunderer]

Interesting, it looks like a leftover from the time I actually used the offset, now replaced with calculation via mb_strlen(). Would you be interested in preparing a PR for this?

[ViliusS]

I've actually tried that, but the logic in the following switch statement is a little bit over my head. What I could share is why I have investigated this, and minimal test cases which shows memory issues with that preg_match_all() statement https://1drv.ms/f/s!AgnMn-haWyFrkcN50beuEO4A0m6PQw?e=GjwcEV . Ideally I wanted to find out why these content cases uses so much memory, but other cases don't. PREG_OFFSET_CAPTURE looked like a low hanging fruit, but there could be other issues still.