Scintillua issue with Lexilla::MakeLexer

[scintillaorg]

Scintillua issue with Lexilla::MakeLexer

[nyamatongwe]

Namespaces are supposed to be optional - you don't have to specify them if you are willing to receive a lexer with that name from any library. If you specifically want the Lua lexer can't you ask for lexer.$(file.patterns.name)=lpeg.name?

[nyamatongwe]

Which library is not providing a namespace? Lexilla as installed into SciTE should be using "lexilla" and Scintillua should be using "lpeg".

[ittegrat]

My use case is the following: I would use the Lua lexer for a private DSL language and the standard Lexilla lexers for all the other files. I encounter the problem using Sc1. As far as I understand, the problem doesn't occur in SciTE because the lexilla.dll is probed before the LexLPeg.dll.
A minimal set of properties to reproduce the problem is the following:

SciTEGlobal.properties

import lexers/lpeg

lexers/lpeg.properties

lexilla.context.lpeg.home=$(SciteDefaultHome)/lexers
lexilla.context.lpeg.color.theme=scite
if PLAT_WIN
  lexilla.path=.;$(lexilla.context.lpeg.home)/LexLPeg.dll
if PLAT_GTK
  lexilla.path=.;$(lexilla.context.lpeg.home)/liblexlpeg.so
fold.by.indentation=0
fold.line.groups=0
fold.on.zero.sum.lines=0

With these settings, I would expect a file, e.g. a python file, to be styled with the static Lexilla lexer set in the embedded properties, but instead, it is styled with a default lpeg lexer.
I tried to to set the namespace, i.e. lexer.$(file.patterns.py)=lexilla.python, but is is not recognized. Does the static lexilla library define the namespace?.

[nyamatongwe]

The statically linked lexers are checked after the dynamically loaded lexers and don't have a namespace. You could grab a copy of Lexilla.dll from a full SciTE download and refer to it in lexilla.path.

There could be a second try of the static call inserting "lexilla." before the name.

[nyamatongwe]

This patch implements a static namespace that refers to statically linked lexers with statements like lexer.*.cpp=static.cpp. That then requires the static. name in other properties style.static.cpp.4=$(colour.number).

diff --git a/access/LexillaAccess.cxx b/access/LexillaAccess.cxx
index f612ce29..2a4b6450 100644
--- a/access/LexillaAccess.cxx
+++ b/access/LexillaAccess.cxx
@@ -234,6 +234,10 @@ Scintilla::ILexer5 *Lexilla::MakeLexer(std::string_view languageName) {
 			}
 		}
 	}
+	constexpr std::string_view builtin("static.");
+	if (HasPrefix(sLanguageName, builtin) && pCreateLexerDefault) {
+		return pCreateLexerDefault(sLanguageName.substr(builtin.size()).c_str());
+	}
 	// If no match with namespace, try to just match name
 	for (const LexLibrary &lexLib : libraries) {
 		if (lexLib.fnCL) {

The static namespace is probed after the dynamically loaded lexer's namespaces but before checking for names without namespaces.