Community
    • Login

    Enable Lexer available in Scintilla

    Scheduled Pinned Locked Moved Help wanted · · · – – – · · ·
    8 Posts 3 Posters 4.6k Views
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • oirfeodent oirfeodentO
      oirfeodent oirfeodent
      last edited by

      Hello Community,
      Scintilla has about 100+ lexers and NPP has around 50+ lexers.
      If a lexer which is present in Scintilla needs to be enabled in NPP… what should be done?
      Is there any documentation available for the same?

      Having an External Lexer could be an option, but wouldn’t that be a overhead to re-implement the same thing done in Scintilla?

      Regards,

      1 Reply Last reply Reply Quote 0
      • dailD
        dail
        last edited by

        Take a look at this issue I created a while ago. It lists all the Scintilla lexer that aren’t being used currently.

        what should be done?

        Take a look at this pull request, that for some reason hasn’t been merged. This is really the minimum requirements to support a new Scintilla lexer.

        1 Reply Last reply Reply Quote 0
        • oirfeodent oirfeodentO
          oirfeodent oirfeodent
          last edited by

          Hi Dail,
          Thx for your response.
          I should create a new merge request similar to the Visual Prolog.
          I should create the next lexer with the define (IDM_LANG + 60) in PowerEditor/src/menuCmdID.h … assuming Visual Prolog would be merged… right?

          Apart from that, PowerEditor/src/ScitillaComponent/ScintillaEditView.h allows LIST_0 to LIST_6 to be passed on to setLexer()… if I remember correctly, Scintilla allows to have 9 WordList by default. Is NPP decision to allow 7 WordList is a design consideration or we can extend it based on the requirement?

          Regards,

          1 Reply Last reply Reply Quote 0
          • dailD
            dail
            last edited by

            assuming Visual Prolog would be merged… right?

            I would not assume it will be merged. It’s been open a long time so I doubt it will get merged soon. Part of the problem with adding new lexers is that every time one is added then it can cause merge conflicts with all the other requests (Just a pain we have to live with though).

            if I remember correctly, Scintilla allows to have 9 WordList by default

            Yes, that is correct.

            Is NPP decision to allow 7 WordList is a design consideration or we can extend it based on the requirement?

            I believe it is because there currently has not been a need. Some minor code modifications would be needed to support having 9 WordLists but shouldn’t be any major problem.

            1 Reply Last reply Reply Quote 0
            • Gur StaviG
              Gur Stavi
              last edited by

              Hi,
              I tried compiling scintillas visual prolog lexer into my external lexer plugin.
              It is a complete hack for now but if it makes anyone happy I may work on it some more someday.
              You can download it here:
              lexamples with prolog
              The lexer should appear as ‘VVV’ in the languages menu.
              I could not test it since I don’t know prolog.
              It seems that comments work.
              You need to set styling colors manually from Setting -> Style configurator and save them.
              Regards,
              Gur

              1 Reply Last reply Reply Quote 0
              • oirfeodent oirfeodentO
                oirfeodent oirfeodent
                last edited by

                @dail , @MAPJe71
                I have created PR #2696 and also added a comment in Issue #1258.

                This PR is for SCLEX_BAAN (BaanC Language). There are new enhancements, for SCLEX_BAAN in Scintilla Version 3.7/3.7.1. I have created the PR now and once NPP’s Scintilla Version gets upgraded, I will create a another PR for enabling the new functionalities.

                Pls, have a look at the PR and let me know the process how/whom should I contact to get the PR committed to master?

                Regards,

                1 Reply Last reply Reply Quote 0
                • dailD
                  dail
                  last edited by

                  @oirfeodent-oirfeodent

                  You’ll have to wait for Don Ho to see it and hopefully merge it. Unfortunately, a few other open PRs adding new lexers have not be accepted in a while.

                  1 Reply Last reply Reply Quote 0
                  • oirfeodent oirfeodentO
                    oirfeodent oirfeodent
                    last edited by

                    @dail
                    Thx for your response.

                    @donho
                    Is there a road map for such requests (Enabling Lexer which is available in Scintilla)?
                    If not also, let us know what you think about it…
                    If any help is required in organizing/executing such stuff as well… let us know.

                    Thx in advance.

                    1 Reply Last reply Reply Quote 0
                    • First post
                      Last post
                    The Community of users of the Notepad++ text editor.
                    Powered by NodeBB | Contributors