[Github-comments] [geany/geany] Add Ocaml ctags parser (PR #3163)

elextr notifications at github.com
Sat Apr 16 01:45:48 UTC 2022


What part(s) does the no apply to?

1. -P doesn't use cpp?
2. one file at a time
3. thats enough for testing

If Geany still copies a single listed source into a temporary file thats indeed silly, so yes fixing that is good.  

Ok, I guess the manual can be read that Geany makes tags for multiple files even with -P but isn't clear the files are catenated.

I see your point that for languages that are not C, catenating them may not make sense.  Changing that, but still putting all the symbols in the one tag file makes sense.  If the user wants catenation they can either do it themselves to a temporary, or not specify -P, let their language be C and use cpp to catenate.  And the name of the variable you changed gives the game away that its intended for languages that are C only :smile: 

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

Message ID: <geany/geany/pull/3163/c1100502403 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20220415/049d16a2/attachment.htm>


More information about the Github-comments mailing list