Neil Hodgson wrote:
The regex code is very simple and was chosen based on licensing
considerations rather than features. IIRC, the RESearch::Execute
method can only search a single line and the calling code in
Document::FindText works around that.
Would it be possible to combine all the lines into a single string (with
linechanges in the middle) and pass this string to the Execute? Or would
this cause major speed or memory losses on large files? I think I might
look into this and test how easy it would be to implement unless you can
tell me reason(s) why not to do it.
I now regard including regular expression support in Scintilla as a
mistake and encourage projects to implement their own version using an
established RE library.
IMHO it would be best if Scintilla would be improved. That would save a
lot of work from open source projects in total as so many projects are
using Scintilla. Also many of the projects have little or no interest in
improving the reg. exp. code as they either see more important problems
or they think that it should be fixed in Scintilla. Also as the search
is already implemented in Scintilla, Scintilla is the logical place for
reg. exp. code also from the point of code readability.
Are there some other problems with the Scintilla reg. exp. or why do you
think the simple reg. exp. code is so bad that people should implement
their own instead of making the Scintilla code better? ( Just curious
whether you think it would be good if someone fixes the multiline issue
or do you think there is no point fixing one issue while so many would
remain. )
_______________________________________________
Scintilla-interest mailing list
[email protected]
http://mailman.lyra.org/mailman/listinfo/scintilla-interest
