Community
    • Login

    Possible error or not?

    Scheduled Pinned Locked Moved Translation
    4 Posts 3 Posters 83 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.
    • U
      Uwo222777
      last edited by

      Possible error
      In one of the previous versions of the English language file there was: <Item id=“7146” name=“MB (1 - 4096)”/>
      Later, a change appeared: <Item id=“7146” name=“MB (1 - 2046)”/>
      Currently, in English and German language files: <Item id=“7146” name=“MB (1 - 2046)”/>
      but in French, as before: <Item id=“7146” name=“Mo (1 - 4096)”/>

      Question: Is this change “correct” or “random”?

      PeterJonesP 1 Reply Last reply Reply Quote 1
      • PeterJonesP
        PeterJones @Uwo222777
        last edited by PeterJones

        @Uwo222777 ,

        The explanation given in the commit message says that the old 4096 Mb limit was actually causing a crash, so it had to be lowered to a limit that was 2046 Mb. That commit changed a bunch of the translation files, but apparently not all of them.

        It was definitely an intentional/correct change to the limit text, and if you’ve found one or more languages that haven’t been updated, that could be reported to the developer using the official Issue interface at the GitHub repo, as described in our feature-request/bug-report FAQ (if it hasn’t been reported already).

        U xomxX 2 Replies Last reply Reply Quote 1
        • U
          Uwo222777 @PeterJones
          last edited by

          @PeterJones
          Thank you very much. Now “everything has fallen into place.”

          1 Reply Last reply Reply Quote 1
          • xomxX
            xomx @PeterJones
            last edited by

            @PeterJones said in Possible error or not?:

            the old 4096 Mb limit was actually causing a crash, so it had to be lowered to a limit that was 2046 Mb.

            It’s true, and the 2046 is the current right ‘Define Large File Size’ threshold max value for the N++ Scintilla syntax highlighting.

            Note:
            Now we could easily shift that back (but I don’t think it’s a good idea, as the enabled syntax highlighting and the assoc. stuff substantially slow down handling and double the memory consumption needed for such large files…) to the previous larger 4096 threshold, since in the meantime I finally persuaded Don and now the SC_DOCUMENTOPTION_TEXT_LARGE Scintilla docs flag is used as default everywhere (this effectively removes the previous crash possibility for a small increase in the consumed memory price). Some details in:
            https://github.com/notepad-plus-plus/notepad-plus-plus/issues/14944
            https://github.com/notepad-plus-plus/notepad-plus-plus/pull/14982

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