[Github-comments] [geany/geany] Remove Ferite parser from c.c (PR #3075)

Jiří Techet notifications at xxxxx
Sat Jan 1 15:11:36 UTC 2022


@techee commented on this pull request.



> @@ -78,7 +78,6 @@ enum
 	TM_PARSER_SH,
 	TM_PARSER_D,
 	TM_PARSER_FORTRAN,
-	TM_PARSER_FERITE,

I just noticed it's not probably OK to do this - it would shift parser identifiers for plugins. We could either keep the dummy `TM_PARSER_FERITE` ID here and instead of `FeriteParser` use whatever other parser in `tm_parsers.h` or replace it with some newly introduced parser (e.g. the GDScript parser or Perl6 parser which I think should be a filetype Geany should add support for).

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

Message ID: <geany/geany/pull/3075/review/842342278 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20220101/4003530c/attachment.htm>


More information about the Github-comments mailing list