Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exception: vector<T> is too long - new cause #151

Closed
aevojoey opened this issue Oct 23, 2018 · 14 comments

Comments

Projects
None yet
5 participants
@aevojoey
Copy link

commented Oct 23, 2018

This seems to be the same error as [https://github.com//issues/139] that was closed on July 22 but with a different way of getting it.

Description of the Issue

Since updating to notepad++ version 7.5.9, I have been getting the "Exception: vector is too long" error. It only occurs when a collapsible line is collapsed and visible on the screen and you try to scroll horizontally. If a collapsed line is not on the screen, there is no error.
I removed the DSpellCheck v1.4.6 plugin and the error stopped.

Steps to Reproduce the Issue

  1. Open a file with collapsible lines (html, js, php, etc.)
  2. Find a line that is longer than the width of the screen with a collapsible line on the screen
  3. Do not collapse the line and scroll horizontally (with home/end, arrows or scroll bar) -> no errors
  4. Collapse a line on the screen and try scrolling horizontally again -> Exception.
  5. Scroll vertically so that the collapsed line is no longer visible and try scrolling -> no error

Debug Information

Notepad++ v7.5.9 (32-bit)
Build time : Oct 14 2018 - 15:02:52
Path : C:\Program Files (x86)\Notepad++\notepad++.exe
Admin mode : OFF
Local Conf mode : OFF
OS : Windows 10 (64-bit)
Plugins : DSpellCheck.dll mimeTools.dll NppConverter.dll NppExport.dll ComparePlugin.dll HTMLTag_unicode.dll ImgTag.dll JSLintNpp.dll JSMinNPP.dll NPPJSONViewer-win32.dll NppTextFX.dll PluginManager.dll XMLTools.dll

@Predelnik Predelnik added this to the 1.4.7 milestone Oct 25, 2018

@Predelnik

This comment has been minimized.

Copy link
Owner

commented Oct 25, 2018

Thank you very much for describing the issue so thoroughly! Since this is a serious problem I will try to release a version with the fix very soon.

Meanwhile could you check if the version of the plugin from the link below fixes the issue for you if it wouldn't be too much trouble?
https://ci.appveyor.com/api/buildjobs/s4mc8o0q6wum0e2i/artifacts/DSpellCheck.dll

@guy038

This comment has been minimized.

Copy link

commented Oct 27, 2018

Hi, Predelnik,

Ah, yes ! aevojoey is quite right about it ! I have this same issue, too and I verified that it happens also, with version v1.4.5 and version v1.4.4 with the fix of the bug #139 :-((

But again, Sergey, the fix of the v1.4.6 that you propose, in case of folded lines, must had been compiled without the XP support ( same issue as in my reply, relative to issue 139 ) !

Many thanks for investigating ;-))

guy038

@Predelnik

This comment has been minimized.

Copy link
Owner

commented Oct 28, 2018

must had been compiled without the XP support

Yeah it was a bug in CI script. This version should be with winxp support:
https://ci.appveyor.com/api/buildjobs/imtsoq6ll9yn8qfe/artifacts/DSpellCheck.dll

@guy038

This comment has been minimized.

Copy link

commented Oct 28, 2018

Hello, Predelnik,

Ah, I'm really sorry, but I always get an error ! This time, the message, in French, said :

The application or DLL C:\WINDOWS\System32\MSVCP140.dll is not a valid Windows image

Oddly, I don't have any dll, named MSVPC140.dll, on my system ! I, then, tried to download, successively, several versions of that dll, from various sites. But, either I got the same message or I had a message that the VCRUNTIME140.dll was missing :-((

And after download of the unique version of VCRUNTIME140.dll that I could find ( among 4 DLL sites ! ), install on C:\windows\system32 and a laptop reboot, I still got errors :-((

What am I doing wrong ?

Cheers,

guy038

@Predelnik

This comment has been minimized.

Copy link
Owner

commented Oct 29, 2018

Yeah it should not be the case, seems to be another problem on my (appveyor) side. This one is a little bit more confusing as it can be seen from logs that it correctly compiles with /MT option for C runtime.

I will look into it.

Edit: it looks like /MD is still present in some compilation lines. Quite possibly it's due to different CMake version.

@Predelnik

This comment has been minimized.

Copy link
Owner

commented Oct 29, 2018

Ok good news is that I've discovered the cause of the issue after spending some time. It was due to bad typo in CMakeLists.txt resulting in my Release build not being correct. And since I deploy 'Release with Debug Info' build most of the time, it was not affecting me that much.

Here is version which should work (sorry for that many iterations):
https://ci.appveyor.com/api/buildjobs/m8jk80d0t0tm5ixv/artifacts/DSpellCheck.dll

@guy038

This comment has been minimized.

Copy link

commented Oct 30, 2018

Hi, Predelnik,

This time , it's the right one, Sergey ! Everything is OK, with your last version, on my Win XP configuration ;-)) No more messages about missing DLL !

Many thanks for checking your code and fixing that issue !

Cheers,

guy038

@trackvsb

This comment has been minimized.

Copy link

commented Nov 13, 2018

I have the same issue when choosing language ASP to view a copy/pasted text. Switching to other languages does not throw this error.

Edit (more info): I had a comment-type line at the top of the copy/pasted text, starting with and including multiple equal sign symbols (=). It was not part of the original ASPX file. Removing this line from the top eliminated the error message.

Windows 7 Professional on desktop, Notepad++ 7.5.4 64 bit.

@Predelnik

This comment has been minimized.

Copy link
Owner

commented Nov 13, 2018

@trackvsb
Just to make sure - is problem still reproducible with the version from the link below?
https://github.com/Predelnik/DSpellCheck/releases/tag/v1.4.7

@trackvsb

This comment has been minimized.

Copy link

commented Nov 14, 2018

The problem disappeared after I updated from 1.4.6 to 1.4.7. Thanks for the fast response!

@Predelnik

This comment has been minimized.

Copy link
Owner

commented Nov 20, 2018

This case should be fixed by 1.4.8 currently available through plugin manager. Please report any other cases of this error, thanks!

@laconeo

This comment has been minimized.

Copy link

commented Mar 4, 2019

i get ---------------------------
Plugin Exception

An exception occurred due to plugin: DSpellCheck.dll
Exception reason: vector too long

Aceptar

every time when i make scroll i get these error

@Predelnik

This comment has been minimized.

Copy link
Owner

commented Mar 4, 2019

Hi @laconeo
First of all which version of the plugin are you using? Can you try the latest version first, you can find here: https://github.com/Predelnik/DSpellCheck/releases/tag/v1.4.10

If it still does not fix the issue it would be really great if you would provide an example of the file where you get this error.

@laconeo

This comment has been minimized.

Copy link

commented Mar 4, 2019

i was using 1.4.6 i have updated to the last version and problem solucionated! thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.