-
-
Notifications
You must be signed in to change notification settings - Fork 10
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
CCVLint 0.4.5 not compatible with Notepad++ 8.4 32-bit #23
Comments
Thanks for the comment, this is the same as issue #22. It's fixed but not yet automatically available in the current version of Notepad++ (it's updated irregularly). However, you can manually install the latest version by downloading the new CSVLint version 0.4.5.1 here and then unzip and replace the CSVLint.dll in your In case there are others with the same issue, I'll leave this issue open until the next Notepad++ update is released. |
Many thanks for replying so quickly. I suspect I've found a bug in 0.4.5.1. I used your download link to get CSVLint_x86.zip. I placed the dll in %ProgramFiles(x86)%\Notepad++\plugins\CSVLint. I opened a .txt file. When I try to apply CSVLint colouring (NPP menu Language > CSVLint), NPP closes immediately. Here are the messages from the Application event log Application: notepad++.exe and Faulting application name: notepad++.exe, version: 8.4.0.0, time stamp: 0x625f61b6 |
That's a bit concerning, thanks for testing the new version like this. I did test the new plugin in Notepad++ 8.4 both 64bit and 32bit by opening some of my test data files and trying some of the options, and simply enabling Syntax Highlighting is the first thing I tried. I suspect it could be a problem with specific data maybe? I know there are still some weird quirks in the code. Thanks for the error message, however that doesn't tell me very much, I need to debug this in Visual Studio. If your file doesn't contains any privacy sensitive data, is it possible to send the data file via e-mail? Or alternatively, if you can find the rows that causes the problem then only send those rows. You can backup the data, delete half of all rows and try again, if it still crashes again delete half all rows etc. Or maybe replace all data with dummy data (see my e-mail in the Help/About box). |
Also it could be this Windows setting maybe. In Notepad++ when you go to |
I've reproduced the error with every file I've tried, all of which were fine with CSVLint 0.4.5 and the previous NPP version. I've emailed a new simple file that also produces the error (it's not specially crafted, as you'll see). Here's the debug info from NPP
|
Here’s a specimen file that produces the failure.
Regards
***@***.***
From: BdR76 ***@***.***>
Sent: 05 May 2022 13:52
To: BdR76/CSVLint ***@***.***>
Cc: ***@***.***>; Author ***@***.***>
Subject: [EXTERNAL] Re: [BdR76/CSVLint] CCVLint 0.4.5 not compatible with Notepad++ 8.4 32-bit (Issue #23)
That's a bit concerning, thanks for testing the new version like this. I did test the new plugin in Notepad++ 8.4 both 64bit and 32bit by opening some of my test data files and trying some of the options.
I suspect it could be a problem with specific data maybe? I know there are still some weird quirks in the code.
Thanks for the error message, however that doesn't tell me very much, I need to debug this in Visual Studio. If your file doesn't contains any privacy sensitive data, is it possible to send the data file via e-mail? Or alternatively, if you can find the rows that causes the problem then only send those rows. You can backup the data, delete half of all rows and try again, if it still crashes again delete half all rows etc. Or maybe replace all data with dummy data (see my e-mail in the Help/About box).
—
Reply to this email directly, view it on GitHub<#23 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AH5VMIVPRJFOFRK33IHTXMDVIO76LANCNFSM5VEVKG2Q>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
"hello","world","!"
"hello","world","!"
|
This issue was with an older version of the plug-in. In the meantime both Notepad++ and the plugin have been updated a couple of times. I assume the Lexer v5 compatibility issues have been fixed now. If any problems still presist then please open a new issue. |
Trying to add CSVLint 0.4.5 through Plugins Admin fails with the following message:
Notepad++ version is 8.4 32-bit
OS is Windows 10 20H2 (OS Build 19042.1645)
The text was updated successfully, but these errors were encountered: