[Github-comments] [geany/geany] Decide what parsers from uctags we can sync with Geany (PR #2991)

[Jiří Techet]

This pull request isn't meant to be merged, it should just serve as a discussion place which parsers apart from those in #2990 to select for syncing. I'm not so familiar with the state of Geany parsers and if there are some patches that should be first applied to uctags before we can grab the uctags version, it would be nice if people who know more than me (@b4n @eht16 @elextr and others) could help with this.

Just to start somewhere, I included a few parsers to this commit:
1. html and go parsers - these are those I initially wrote and I think their development happened in uctags
2. diff, jscript, make, objc, rust - there don't seem to be too many changes in these parsers and the uctags version seems to be a superset of the geany version

I went through all the parsers and divided them into 2 categories:

Parsers where uctags implementation is similar to the Geany implementation:
- asm
- basic
- diff (already included)
- fortran
- go (already included)
- html (already included)
- jscript (already included)
- lua
- make (already included)
- objc (already included)
- pascal
- perl
- php
- ruby
- rust (already included)
- sql
- nsis

Parsers which differ a lot:
- python (but here I assume the uctags version is the Colomban's new python parser we could grab)
- iniconf
- matlab
- r
- sh
- tcl
- tex
- verilog
- vhdl
- and of course all the languages from c.c

Apart from python, the languages from the first group seem to be the most obvious candidates for syncing. So does anyone have a better knowledge of the state of these parsers?

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

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

-- Commit Summary --

  * <a href="https://github.com/geany/geany/pull/2991/commits/8ab2e0e68ce33fd0a883c8f4f778bf6233882dc9">Remove the geany_ prefix from parsers that seem to be good candidates for syncing</a>
  * <a href="https://github.com/geany/geany/pull/2991/commits/425d350014e065f79eb2af863e0516d0cb1d26a5">Copy over selected parsers from uctags so we can see the diffs</a>

-- File Changes --

    R ctags/parsers/diff.c (93)
    D ctags/parsers/geany_go.c (831)
    A ctags/parsers/go.c (1416)
    R ctags/parsers/html.c (168)
    R ctags/parsers/jscript.c (462)
    R ctags/parsers/make.c (171)
    R ctags/parsers/objc.c (333)
    R ctags/parsers/rust.c (95)

-- Patch Links --

https://github.com/geany/geany/pull/2991.patch
https://github.com/geany/geany/pull/2991.diff

-- 
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20211108/d4a57dd8/attachment-0001.htm>