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

Hard crash of 32bit NPP when formatting a single line of JSON #152

Closed
boris-v opened this issue May 28, 2024 · 2 comments
Closed

Hard crash of 32bit NPP when formatting a single line of JSON #152

boris-v opened this issue May 28, 2024 · 2 comments

Comments

@boris-v
Copy link

boris-v commented May 28, 2024

Windows, NPP latest version, 32bit. Latest plugin version (1.2312).
The bug is only with 32bit NPP; not reproducible on 64bit npp (same version of NPP & plugin).

Test JSON line:

[{"i": null, "j": null, "k": null, "l": "P1", "m": false, "n": "a", "o": "szdf", "p": 123456}, {"i": null, "j": null, "k": null, "l": "P2", "m": false, "n": "a", "o": "szdf", "p": 123456}, {"i": null, "j": null, "k": null, "l": "P3", "m": false, "n": "a", "o": "szdf", "p": 123456}, {"i": null, "j": null, "k": null, "l": "P4", "m": false, "n": "a", "o": "szdf", "p": 123456}, {"i": null, "j": null, "k": null, "l": "P5", "m": false, "n": "a", "o": "szdf", "p": 123456}, {"i": null, "j": "qwerty", "k": "zxcvb", "l": "P1", "m": false, "n": "a", "o": "szdf", "p": 123456}, {"i": null, "j": "qwerty", "k": "zxcvb", "l": "P2", "m": false, "n": "a", "o": "szdf", "p": 123456}, {"i": null, "j": "qwerty", "k": "zxcvb", "l": "P3", "m": false, "n": "a", "o": "szdf", "p": 123456}, {"i": null, "j": "qwerty", "k": "zxcvb", "l": "P4", "m": false, "n": "a", "o": "szdf", "p": 123456}, {"i": null, "j": "qwerty", "k": "zxcvb", "l": "P5", "m": false, "n": "a", "o": "szdf", "p": 123456}, {"i": null, "j": null, "k": "zxcvb", "l": "P1", "m": false, "n": "a", "o": "szdf", "p": 123456}, {"i": null, "j": null, "k": "zxcvb", "l": "P2", "m": false, "n": "a", "o": "szdf", "p": 123456}, {"i": null, "j": null, "k": "zxcvb", "l": "P3", "m": false, "n": "a", "o": "szdf", "p": 123456}, {"i": null, "j": null, "k": "zxcvb", "l": "P4", "m": false, "n": "a", "o": "szdf", "p": 123456}, {"i": null, "j": null, "k": "zxcvb", "l": "P5", "m": false, "n": "a", "o": "szdf", "p": 123456}]
  1. Paste the JSON into a blank npp document, single line.
    image
  • Plugin JSTool -> JSFormat: formats JSON
    (Works correctly)
    image

  • Select all, with the entire line selected (but it must be a single line):
    image
    apply JSFormat: Causes NPP to crash hard

  1. It works correctly with the selection if a single newline is added to the end.
    Paste the JSON into a blank document, single line, add one newline (Unix - LF), Select All.
    image
    Plugin JSTool -> JSFormat with this JSON line + newline selected:
    image
@sunjw
Copy link
Owner

sunjw commented May 29, 2024

I couldn't reproduce this issue. You can see the screen record I attached. Is there any step wrong in the record?

Video_2024-05-29_100016.mp4

@boris-v
Copy link
Author

boris-v commented May 29, 2024

Thanks for looking into it!
I cannot reproduce it anymore, either 😓

I only realized I still had the 32bit version when making this issue, and I replaced it with 64bit afterwards. I now reinstalled the 32bit version, installed the plugins, and could not reproduce it again.

Anything more is just guessing... It might be that something went wrong with one of the updates, I've had that 32bit installation for many years and I just kept upgrading NPP and the plugins.

@boris-v boris-v closed this as completed May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants