In addition to JSON, the other lexer we don't use yet is for the Nim language (these are the only ones I'm aware of). I'm not a Nim language user myself but I tried to modify the external filetype config file to the internal one so most of the stuff is preserved and should work correctly. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/3648
-- Commit Summary --
* Convert Nim to builtin filetype and use Scintilla Nim lexer for it
-- File Changes --
M data/Makefile.am (2) D data/filedefs/filetypes.Nim.conf (64) A data/filedefs/filetypes.nim (71) M data/filetype_extensions.conf (2) M meson.build (1) M scintilla/Makefile.am (1) A scintilla/lexilla/lexers/LexNim.cxx (814) M scintilla/lexilla/src/Lexilla.cxx (1) M scintilla/scintilla_changes.patch (1) M src/filetypes.c (1) M src/filetypes.h (1) M src/highlighting.c (15) M src/highlightingmappings.h (29)
-- Patch Links --
https://github.com/geany/geany/pull/3648.patch https://github.com/geany/geany/pull/3648.diff
In addition to https://github.com/geany/geany/pull/3171 I'd like to merge this PR too so there's a proper lexer for the Nim language. I just went through the PR and tested it a bit and all seems to work well.
Again, I'll wait for about a week for complaints and merge this afterwards.
(Misusing hot summer days where nobody pays attention to get some old patches of mine merged :-P.)
Conflicts need resolving.
Its meant to be long __lazy__ summer days :stuck_out_tongue:
Conflicts need resolving.
I'm aware of that. But because I'll introduce other conflicts by merging https://github.com/geany/geany/pull/3171, I'll wait until that one is merged.
Its meant to be long lazy summer days 😛
Yeah, just merging stuff is lazy :-)
https://github.com/nim-lang/langserver
By the way I tried that one but it kept crashing for me and I didn't spend much time investigating what went wrong.
but it kept crashing for me and I didn't spend much time investigating what went wrong.
Ok, was just a suggestion since there is no ctags parser.
@techee pushed 1 commit.
3c6021ee6614d987b7b6bfcb4eab1c049b067732 Convert Nim to builtin filetype and use Scintilla Nim lexer for it
Merged #3648 into master.
github-comments@lists.geany.org