@cousteaulecommandant commented on this pull request.


In data/filedefs/filetypes.systemverilog:

> @@ -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, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <geany/geany/pull/4039/review/2425876850@github.com>