search

BdR76 / CSVLint

CSV Lint plug-in for Notepad++ for syntax highlighting, csv validation, automatic column and datatype detecting, fixed width datasets, change datetime format, decimal separator, sort data, count unique values, convert to xml, json, sql etc. A plugin for data cleaning and working with messy data files.
GNU General Public License v3.0
151 stars 8 forks source link

Unhandled Exception #65

Closed Friedi closed 1 year ago

Friedi commented 1 year ago

Notepad++: 8.5.2 (x64) CSVLint: 0.4.6.3

When I open the following non csv conform text file and do "detect columns" (with automatic unchecked), and hitting "ok" in the manual detection dialog, Notepad++ crashes with an infinite loop of unhandled exceptions. Only killing the notepad task closes it.

Preconditioning_working.csv CVInt Error

BdR76 commented 1 year ago

Thanks for posting the issue, but I can't reproduce this.

The file encoding in the bottom right says UTF-8 so that should be ok, maybe it has to do with the Windows codepage setting? Can you post the text from the ? > Debug info menu?

Friedi commented 1 year ago

I see my description was not correct, sorry for that. The error occurs only if you click "detect columns" (unchecked "automatic") and then "ok" in the dialog for manual detection.

see attached text file for the error msg debug.txt

The exception also occurs for a pure notepad++ and just this plugin.

the debug.info from notepad++:

Notepad++ v8.5.2 (64-bit) Build time : Apr 4 2023 - 19:55:32 Path : C:...\Notepad++\notepad++.exe Command Line : Admin mode : OFF Local Conf mode : ON Cloud Config : OFF OS Name : Windows 10 Enterprise (64-bit) OS Version : 22H2 OS Build : 19045.2728 Current ANSI codepage : 1252 Plugins : AnalysePlugin (1.13.49) BetterMultiSelection (1.5) BookmarksDook (2.3.5) ColumnTools (1.4.5.1) ComparePlus (1.1) CSVLint (0.4.6.3) CustomLineNumbers (1.1.8) DSpellCheck (1.5) ElasticTabstops (1.3.2) ExtSettings (1.3.1) FWDataViz (2.6.2) HexEditor (0.9.12) MarkdownViewerPlusPlus (0.8.2) mimeTools (2.9) NavigateTo (2.4.1) NppAutoDetectIndent (2.3) NppBplistPlugin (1.3) NppConverter (4.5) nppcrypt (1.0.1.6) NppExport (0.4) NppFavorites (1.0.0.1) NPPJSONViewer (2.0.4) NppMarkdownPanel (0.7.3) NppQrCode (0.0.0.1) NppRegExTractorPlugin (2.1) NppTaskList (2.5) NppTextViz (0.4.2) NppToolBucket (1.10.6622.41516) NppXmlTreeviewPlugin (2) qkNppReverseLines (1) QuickText (0.2.5.1) SessionMgr (1.4.4) XBrackets (1.3.1) XMLTools (3.1.1.13) _CustomizeToolbar (5.3)

BdR76 commented 1 year ago

Oops can't believe I missed that one.. This has to do with the DetectColumnWidths setting, the bug happens when the "Columns positions" text box is empty. Work-around is to start the manual detect dialog, for one time select separator as {fixed width} and then enter anything like 1,2,3 in the "Columns positions" textbox and press OK. After that it should work again without crashing.

The issue is fixed now and will be available in the next release

BdR76 commented 1 year ago

@Friedi fyi I've updated the development DLL from here which includes this fix. If you are able to try this version then let me know if there are any other issues

(edit: I just found out that in the development build the "Analyze data report" is broken, I'll fix that)

Friedi commented 1 year ago

@Friedi fyi I've updated the development DLL from here which includes this fix. If you are able to try this version then let me know if there are any other issues

no issues with that anymore thanks!