[Github-comments] [geany/geany] Sync ctags parsers with big changes with Geany (PR #2991)

Enrico Tröger notifications at xxxxx
Sun Nov 28 10:52:37 UTC 2021


I had a quick look on the changes and this looks good, we'll get some more new tag types and probably the newer uctags code is less buggy than the current ones.

Though at least in the Pascal parser we will lose two features: 
- parsing "type" definitions (IIRC "type" in Pascal is somewhat similar to "structs" in C)
- parsing argument list of functions

This two features are not in the uctags Pascal parser. I don't know where they came from, maybe we added them ourselves in the past.

Since these could be useful for others as well, it might be best to push these two upstream to uctags, so we'll get them back with the next uctags update.

Until then, it might be ok to have the two features removed (which will also solve #2987) by this PR and get them back with the next uctags sync.

An alternative could be to remove the Pascal parser from this PR and merge #2987 in the meantime.

I prefer the first option: use the uctags Pascal parser with this PR.

What do you think?


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2991#issuecomment-981063263
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20211128/ad141542/attachment.htm>


More information about the Github-comments mailing list