Fixes #2358, fixes #2982, fixes #2428 and fixes #2987.
After our `parseArglist` implementation has been accepted upstream (and improved), this PR will now bring back the unmodified Pascal parser from uctags to Geany!
This will **remove** our buggy, incomplete implementation of parser `type` declarations which caused crashes and never worked really well. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/3043
-- Commit Summary --
* Use Pascal parser from uctags
-- File Changes --
M ctags/Makefile.am (2) R ctags/parsers/pascal.c (98) M tests/ctags/bug612019.pas.tags (2)
-- Patch Links --
https://github.com/geany/geany/pull/3043.patch https://github.com/geany/geany/pull/3043.diff
Nice! It turned out to be quite a project when looking at the upstream pull request. Happy you picked the work first :-).
Happy that you picked so many of the other parsers!!
@eht16 pushed 1 commit.
f04fbab556048fb4706c582e1d2440fe758f117a Merge branch 'master' into pascal_sync
@eht16 pushed 1 commit.
683c846873461d35f3deeabb7648a5d09bc73050 Use Pascal parser from uctags
Updated the Pascal parser again from uctags as it recently changed (more fixes).
I went briefly through the code changes and there doesn't seem anything bad to me. Since this pull request fixes a crash, I'm merging it (and giving others a positive example what to do with my pull requests ;-).
Merged #3043 into master.
github-comments@lists.geany.org