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

NUL character #1446

Closed
Nemo271 opened this issue Jul 19, 2019 · 11 comments
Closed

NUL character #1446

Nemo271 opened this issue Jul 19, 2019 · 11 comments

Comments

@Nemo271
Copy link

Nemo271 commented Jul 19, 2019

When I open file *.csv (in Notepad3 v5.19.630.2381) between each character I have NUL character.
Screen Shot 19 07 2019 at 21 35

On version Notepad3 5.19.108.1602 (and old) was correct
Screen Shot 19 07 2019 at 21 36

@RaiKoHoff
Copy link
Collaborator

@Nemo271 Please attach an example file (.zip packed) to this issue.

@Nemo271
Copy link
Author

Nemo271 commented Jul 20, 2019

This is example file (.zip packed).
test.zip

@hpwamr
Copy link
Collaborator

hpwamr commented Jul 20, 2019

Hello @RaiKoHoff , here the result of a first analyze:

"test.cvs' (the file is "UCS-2 LE")

  • Notepad3 v5.19.719.2399 BETA "Out-of-the-Box" --> showed in statusbar: UTF-8 👎
  • Notepad3 v5.19.501.1685 RC "Out-of-the-Box" --> showed in statusbar: UTF-8 👎
  • Notepad3 v5.19.108.1602 "Out-of-the-Box" --> showed in statusbar: UTF-16 LE 👍
  • Notepad++ 7.7.1 "Out-of-the-Box" --> showed in statusbar: UCS-2 Little Endian 👍

Details:

  • Notepad3 v5.19.719.2399 BETA "OTB", UCHARDET in Debug Mode: 1
    UCD=ASCII 100% (reliable), OS (CP-1252), showed in statusbar: ANSI (CP-1252) 👎
  • Notepad3 v5.19.501.1685 RC "OTB", UCHARDET in Debug Mode: 1
    UCD=ASCII 100% (reliable), CED=UTF-16LE, Conf= 20% (NOT reliable),
    OS (CP-1252) showed in statusbar: ANSI (CP-1252) 👎

My conclusion:

  • The file is coded in an older obsolete "UCS-2 LE" (Unicode UTF-16 LE without BOM).
  • Our current "UCHARDET fails to detect "UCS-2 LE" 🤔

From Unicode Technical Site:

@RaiKoHoff
Copy link
Collaborator

RaiKoHoff commented Jul 21, 2019

Oh, the first case, where deprecated CED has been better 🤔.
Ed.: The issue has been discussed here:

@hpwamr
Copy link
Collaborator

hpwamr commented Jul 21, 2019

Hello @Nemo271 ,

Feel free to test the Beta version "Notepad3Portable_5.19.721.2402_BETA" or higher.
See "Notepad3 beta-channel access #1129" or download it from Notepad3Portable_5.19.721.2402_BETA.

@Nemo271
Copy link
Author

Nemo271 commented Jul 21, 2019

Hello!
On this beta version all is OK , thx.
Now I will wait for setup version for replacing the default Windows Notepad.

@hpwamr
Copy link
Collaborator

hpwamr commented Jul 21, 2019

Hello @Nemo271 , Just for your info:
You can use the installer version Notepad3_5.19.630.2381_Setup version to configure the default Windows Notepad override.
Then, following the instructions of #1105, you will be able to pass to the last version "BETA". 🤔

@hpwamr
Copy link
Collaborator

hpwamr commented Jul 21, 2019

On this beta version all is OK , thx.

As far as I am concerned, this issue may be closed....

@Nemo271
Copy link
Author

Nemo271 commented Jul 22, 2019

OK.
Thx for instructions of #1105

@hpwamr
Copy link
Collaborator

hpwamr commented Sep 27, 2019

Hello @Nemo271 ,
As far as I'm concerned, I think you (as requester) can close this issue...

@Nemo271 Nemo271 closed this as completed Sep 27, 2019
@Nemo271
Copy link
Author

Nemo271 commented Sep 27, 2019

Is ok now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants