Hello Arash,
On 19/02/2023, Arash Esbati wrote:
> Thanks for checking. Can you please elaborate what you mean the "a bit
> inconsistent"? I'm not sure IIUC.
Sorry, I meant the behaviour subsequent I described subsequently.
> So the braces around the argument of \href have the syntax class "string
> fence". And possible backslashes at last chars get the Punctuation
> class so they don't mess the fontification by escaping the last }. This
> is working as expected.
Ah OK, I am not sure why the braces have a different class, but as long as it
is intended.
> Yes, this part of the patch
>
> + "\\(?:{[^}{]*}[^}{]*?\\)*"
>
> means to match only one level of subbraces, and not 2 or more. Is there
> a use-case for this? We can change it, but in general it makes matching
> a little slower (I'm not sure if it has a significant impact on speed
> though).
No particular use-case, just was wondering if it was intended.
Best regards,
-- Al
