vis icon indicating copy to clipboard operation
vis copied to clipboard

filetype.lua refers unknown lexers (ie. modula2)

Open lanodan opened this issue 4 months ago • 4 comments

Problem

$ vis foo.def

Triggers the following backtrace in a window:

/usr/share/vis/lexer.lua:1522: no file '/usr/bin/lua/lexers/modula2.lua'
no file '/usr/bin/lua/modula2/init.lua/modula2.lua'
no file '/home/haelwenn/.config/vis/lexers/modula2.lua'
no file '/home/haelwenn/.config/vis/modula2/init.lua/modula2.lua'
no file '/etc/vis/lexers/modula2.lua'
no file '/etc/vis/modula2/init.lua/modula2.lua'
no file '/usr/share/vis/lexers/modula2.lua'
no file '/usr/share/vis/modula2/init.lua/modula2.lua'
no file '/usr/share/lua/5.4/lexers/modula2.lua'
no file '/usr/share/lua/5.4/modula2/init.lua/modula2.lua'
no file '/usr/lib/lua/5.4/lexers/modula2.lua'
no file '/usr/lib/lua/5.4/modula2/init.lua/modula2.lua'
no file '/modula2.lua'

Steps to reproduce

No response

vis version (vis -v)

vis 0.9 +curses +lua +acl

Terminal name/version

No response

$TERM environment variable

No response

lanodan avatar Aug 27 '25 20:08 lanodan

Cannot reproduce with the master vis. Probably got fixed somewhere in these commits:

16e31ceb lua/lexers: update to scintillua 6.5
0208d7a9 update to scintillua-6.4 + 1 (org-mode lexer)
10d045c4 Add Nix lexer
016d58d8 Add Factor lexer
e54caf08 Rename 'ansi_c', 'dmd', and 'rstats' lexers to 'c', 'd', and 'r'
c1f4d3f6 lexers: switch to tabs for indentation
b9fb1c6c Update lexer LDoc
603bc09d Increase the default LPeg stack size
8e9c7655 pull latest changes from scintillua

mcepl avatar Aug 28 '25 22:08 mcepl

Hmm, Even if it wasn't, an error is unnecessary, it would be better to just vis:info() in case of a missing lexer and fall back to text.

Nomarian avatar Sep 16 '25 14:09 Nomarian

Was fixed in https://github.com/martanne/vis/commit/a7aac1044856abc4d1f133c6563fc604d7fe6295

milhnl avatar Nov 10 '25 13:11 milhnl

Also, it is not completely unknown, https://github.com/paaguti/textadept-modula-lexers … but nobody bothered with porting them to the modern Scintillua style and work with orbitalquark to get them into Scintillua.

mcepl avatar Nov 11 '25 20:11 mcepl