@cousteaulecommandant commented on this pull request.


> @@ -0,0 +1,62 @@
+# For complete documentation of this file, please see Geany's main 
documentation

Oookay, that makes sense.  That's how I initially thought this was meant to be 
done, but eventually I mistakenly understood that the only way to add 
SystemVerilog parsing was as a separate builtin language.  Overall, I had a 
hard time figuring out what needed to be added and what would be reused.

So in summary, internally there are file types (lexer, GEANY_FILETYPES, 
highlighting, etc) and tag managers (ctags, TM_PARSER, etc).  I should nix the 
former and keep the latter, correct?  I thought they were somewhat related.

So the only changes in C/C++ code I get to keep are:
1. Adding SystemVerilogParser
2. Adding the stuff I added to tm_parser

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

Message ID: <geany/geany/pull/4039/review/[email protected]>

Reply via email to