> Maybe I should add a few extra lines to that test with some of the new corner 
> cases I'm now capturing, like the case with = in a comment.

Certainly more is always better, in case someone else breaks it at some point 
:-)

> As per the level of support for regular expressions, I see no other ctags 
> parser using tagRegexTable, so I'm not sure it's supported. (But even if it 
> is, I'm not sure it's a good idea to use it only for Matlab.)

Yeah, as per the comment in #3170 there are no others, and there is a common 
apprehension that regex parsers are slow, so having them run between keystrokes 
is an issue.

> I notice that other tests have a much more complex .tags file which seems to 
> include an argument list,

You are running up against the limits of my knowledge of ctags/tagmangler (the 
expert referred to previously is _NOT_ me) but IIUC the arglist is a separate 
field on the tag, its not part of the tag name (for the reasons you made this 
PR).  Not sure how that is added, maybe look at a parser that does it?

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3358#issuecomment-1364809047
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3358/[email protected]>

Reply via email to