Marc André Tanner

Results 102 comments of Marc André Tanner

Thanks for the report. I briefly looked into the issue and the problem is that we don't actually wait for process termination, but until all connected pipes to the standard...

Thanks for bringing this up. I initially chose Scintillua because it is flexible, relatively lightweight, easy to integrate and had a lot of existing syntax definitions. Basically I was looking...

> If I understand it correctly tracking the new upstream format used by Scintillua isn't an option for performance reasons No, that is not what I intended to express. I...

I rebased our changes on top of the most recent upstream `lexer.lua` and pushed the result to the [scintillua branch](https://github.com/martanne/vis/tree/scintillua). It isn't really tested, but at least in theory should...

> Note that both Geany and Anjuta actually use Scintilla as well I am not really familiar with either environment, but don't they typically use the C++ lexers? > goes...

The current effort can be tracked in the [scintillua](https://github.com/martanne/vis/tree/scintillua) branch. The remaining TODO items are: - [ ] theme review: some lexers use pre-defined styles which our (default) themes should...

I'm fairly new to the "Lua Universe" and thus probably do not know about some of the subtler differences/incompatibilities among the language versions. How is the general ecosystem? Which version...

The recently added [code to handle digraphs](https://github.com/martanne/vis/blob/007ff80b477880e820d08391b96ca2a00889082e/lua/vis-std.lua#L149) uses `file:close()` to get the exit status of an `io.popen`-ed sub process. This functionality isn't available in standard Lua 5.1.

A couple of comments: * Can you use pure lpeg patterns instead of relying on the `re` module? You can also reuse some [patterns provided by the lexer module](https://github.com/martanne/vis/blob/master/lua/lexers/lexer.lua#L1303). *...

I think you misunderstand how `match_at` is supposed to work (and yes the documentation probably sucks). It will start at `pos - horizon` and repeatedly attempt to match the pattern...