Menu

#1324 Scintilla lexer for Hollywood

Committed
closed
nobody
5
2019-12-07
2019-11-10
afalkenhahn
No

Here's a lexer that adds support for Hollywood scripts (.hws) to Scintilla. The patch is against the latest Scintilla version 4.2.1. See here for more information on Hollywood: https://en.wikipedia.org/wiki/Hollywood_(programming_language)

1 Attachments

Discussion

  • Neil Hodgson

    Neil Hodgson - 2019-11-10

    This appears a little outdated with ILexer instead of the current ILexer4. lvOriginal -> lvRelease4. It also has an #ifdef SCI_NAMESPACE to use the namespace but SCI_NAMESPACE is no longer set and the namespace is always on.

    The quickest way to update a lexer to ILexer4 with simple or no-op implementations of the additional methods is to derive from DefaultLexer instead of ILexer and include the DefaultLexer header. See LexPython.cxx for an example.

    There are also a couple of unnecessary elements: styleBeforeKeyword and IsLetter.

     
  • afalkenhahn

    afalkenhahn - 2019-11-11

    Ok, I've fixed this now. New patch attached. I'm still on Scintilla 3.7.2 because that is the version that is shipped with wxWidgets. Even the latest version of wxWidgets from two weeks ago doesn't come with a newer version of Scintilla :-( So this explains why my lexer used an obsolete interface because I'm using Scintilla through wxWidgets.

     
  • Neil Hodgson

    Neil Hodgson - 2019-11-11
    • labels: patch --> patch, scintilla, lexer, hollywood
    • Group: Completed --> Committed
     
  • Neil Hodgson

    Neil Hodgson - 2019-11-11

    Committed as [abba8b].

     

    Related

    Commit: [abba8b]

  • Neil Hodgson

    Neil Hodgson - 2019-12-07
    • status: open --> closed
     
  • Neil Hodgson

    Neil Hodgson - 2019-12-07

    Committed as [abba8b].

     

    Related

    Commit: [abba8b]


Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.