Parsers for R and verilog from ctags are token-based instead of our line-based parsers and are definitely superior and all development should go towards these parsers.

The sh parser is not token based but the uctags version seems to handle more things.

Other 2 parsers which I think we should eventually adopt are VHDL and tex parsers (both are token-based) - they generate a slightly different set of tags and tag mappings should be adopted to those for the sidebar so I skipped them for now.

Finally, I tried also switching to uctags c.c for java, D and C# but our c.c seems to be actually better there.

You can view, comment on, or merge this pull request online at:

  https://github.com/geany/geany/pull/3035

Commit Summary

• cf7f9b3 Use uctags R parser
• e4416f2 Update R kind mappings
• ceddba0 Use uctags parser for sh
• 55b1668 Update mappings for sh parser
• 433d8dc Use uctags verilog parser
• 13d389d Update mappings for verilog parser
• 205c7d3 Update verilog unit tests
• 81d6afd Add R and Verilog to languages reporting context

File Changes

(11 files)

• M ctags/Makefile.am (7)
• D ctags/parsers/geany_r.c (177)
• D ctags/parsers/geany_sh.c (110)
• D ctags/parsers/geany_verilog.c (332)
• A ctags/parsers/r.c (1470)
• A ctags/parsers/r.h (111)
• A ctags/parsers/sh.c (490)
• A ctags/parsers/verilog.c (2024)
• M src/tagmanager/tm_parser.c (14)
• M tests/ctags/bug1111214-j-chan.v.tags (4)
• M tests/ctags/traffic_signal.v.tags (24)

Patch Links:

• https://github.com/geany/geany/pull/3035.patch
• https://github.com/geany/geany/pull/3035.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.