Notepad++ 7.9 RC

donho

How did you install NPP_HexEdit v0.9.6?
Could you also provide the latest version of Notepad++ in which NPP_HexEdit v0.9.6 works ?

deus ex

Hi @donho,

thank you for getting back to me on short notice.

I use a local setup (DoLocalConf.xml) with all plugins located in the subfolder ‘Plugins’ along with nppPluginList.dll. Like I mentioned above, the current full release version 7.8 (32-bit) works fine with NPP_HexEdit v0.9.6 (32-bit), and so did all previous releases of Notepad++. My OS is Windows 10 64-bit build 2004

Please let me know if you need any more details or perhaps would like me to run a debug version, or whatever comes to mind.

P.S.: I just tried the previous release of NPP_HexEdit v0.9.5.19 with the NP++ release candidate, giving the same result (access violation).

Best regards.

donho

@deus-ex
Sorry, I wasn’t clear on my question.

I cannot compile NPP_HexEdit v0.9.6 so I use PluginAdmin of 7.9 RC to download NPP_HexEdit v0.9.5 - execption message as you mentioned.

I installed Notepad++ v7.8.1 (32 bits), and NPP_HexEdit v0.9.5 works for me.

So I need NPP_HexEdit v0.9.6 binary but I have no idea how to get it.
And also I need the accurate info for knowing on which version it doesn’t work anymore - this info helps me to know from which version HexEdit doesn’t work so even maybe I cannot help, I may be able to provide the info to maintainer for fixing it.

ArkadiuszMichalski

@donho Not from here https://github.com/chcg/NPP_HexEdit/releases?

ArkadiuszMichalski

Btw., I also use HexEditor 0.9.5.19 in NPP 7.8 and it’s work, some changed in NPP 7.9 break HexEditor 0.9.5.19 (including 0.9.6). Version 0.9.5 in plugin list is really outdated and probably should be change to chcg fork (after solving this problem).

deus ex

@donho

Like ArkadiuszMichalski kindly noted you can obtain compiled binaries of NPP_HexEdit from the dedicated Releases page.

@ArkadiuszMichalski
Thank you for adding to the discussion.

Best regards.

Alan Kilborn

@donho said in Notepad++ 7.9 RC:

Add Escape keypress in functionlist searchbox clears its text.

I noticed for this change, which works to clear the text, it also sounds a bell?
Not a huge deal, but probably unintended.
I looked at the code for the change, and it seems reasonable; not sure why it would sound a bell. But, I’m far from a Win32 programming expert.

Otherwise, there seem to be a lot of great enhancements here.
Unmentioned directly is the new Searching section in the Preferences, where the following new choices are located:

1. Add confirmation preference for Replace All in All Opened Documents operation.
1. Make (Find) dlgAlwaysVisible a true Preference setting.

For years(!) I’ve wondered why 22 had to be done with a hack; good to see that it is hack-no-longer-required

I also like 9. Add ability to sort lines randomly ! It will be very handy!

donho

@deus-ex @ArkadiuszMichalski

Could you try this build (based on @chcg 's 0.9.6 32 bits) and confirm me if it’s fixed with Notepad++ v6.9 RC ?
http://download.notepad-plus-plus.org/repository/MISC/HexEditor.zip

@chcg I notice HexEditor in PluginList is not point to your release. Is there any reason for not point to it ?

donho

@Alan-Kilborn

You may want to thank @Scott-Sumner :)

ArkadiuszMichalski

@donho
I can confirm that your fix HexEdit version 0.9.6 works in Notepad++ 7.9 RC. Problem is that Plugin Admin serve older 0.9.5 and probably a lot of people will get a crash after upgrading to 7.9, and may not know how to fix it manually. I don’t know how you solve this type of problem. << edit, hmm, not crash, just dialog box so no problem

PS, which of the news broke the HexEditor plugin?

deus ex

@donho said in Notepad++ 7.9 RC:

@deus-ex @ArkadiuszMichalski

Could you try this build (based on @chcg 's 0.9.6 32 bits) and confirm me if it’s fixed with Notepad++ v6.9 RC ?
http://download.notepad-plus-plus.org/repository/MISC/HexEditor.zip

I also can confirm that your HexEditor version runs fine with 7.9 RC, thank you. I assume you forward your fixes to the NP_HexEditor plugin repository?

deus ex

@ArkadiuszMichalski said in Notepad++ 7.9 RC:

@donho
I don’t know how you solve this type of problem. << edit, hmm, not crash, just dialog box so no problem

Actually it is a problem as the original plugin simply won’t run with Notepad++ 7.9 RC. Note, that the 64-bit version of the plugin is affected likewise.

Alan Kilborn

@donho said in Notepad++ 7.9 RC:

You may want to thank @Scott-Sumner :)

Not sure I understand this.
Of course I wish to thank ALL N++ developers bringing us this new release!
Well done, a lot of nice new “toys”, and some old “toys” have been made better!

Or maybe what is meant is that I should thank (blame) a specific dev for ringing the bell on me in the Function List ? :-)

donho

@ArkadiuszMichalski @deus-ex
Thank you for the test & the confirmation.

I assume you forward your fixes to the NP_HexEditor plugin repository?

Yes, sure. Just did a PR for @chcg to check and merge it:
https://github.com/chcg/NPP_HexEdit/pull/37
And I will include a new pluginList probably, if @chcg is OK to do a new release of HexEdit and point to it in the pluginList.

I don’t know how you solve this type of problem.

As mentioned in PR, HexEdit try to get the shortcut of commands from Notepad++, the wrong logic (the bug) in HexEdit worked until Notepad++ removes some shortcuts from the latest version.

Actually it is a problem as the original plugin simply won’t run with Notepad++ 7.9 RC. Note, that the 64-bit version of the plugin is affected likewise.

The fix will apply to both 32 and 64 bits version.

donho

@Alan-Kilborn said in Notepad++ 7.9 RC:

Of course I wish to thank ALL N++ developers bringing us this new release!

Yes, thank all people who have contributed to this project.

Not sure I understand this.

9 19 20 22 of change log (and lots more) are the implementation from @Scott-Sumner.
BTW, I didn’t really follow what you said:

I noticed for this change, which works to clear the text, it also sounds a bell?

Now I get it by hearing the bell.
I just checked the code, but cannot see what it’s wrong. If people complain then we will see what we can do about it.

deus ex

@donho

Hi don,

thank you for fixing the NppHexEdit plugin issue, it’s very much appreciated.

With regards to changelog item no. 9, add ability to sort lines randomly, there’s actually one reason that prevents me from migrating from Notepad++ 32-bit to 64-bit. I’m using the plugin NppTextFX for over a decade now just for ~~two~~ one of its functions. One was the ability to ‘trim trailing white space’ which eventually found its way into Notepad++, and I’m using the internal solution ever since.

The only other function that is important to me currently only available in NppTextFX is to ‘sort lines case insensitive’. Yet the plugin is only available in 32-bit. Would you be able to implement this into Notepad++?

donho

@Alan-Kilborn said in Notepad++ 7.9 RC:

I noticed for this change, which works to clear the text, it also sounds a bell?

Fixed and committed:
https://github.com/notepad-plus-plus/notepad-plus-plus/commit/f3fce2de4a5bf17143af9df31e67f4162d215f70

Thank you for the heads up!

donho

@deus-ex said in Notepad++ 7.9 RC:

The only other function that is important to me currently only available in NppTextFX is to ‘sort lines case insensitive’. Yet the plugin is only available in 32-bit. Would you be able to implement this into Notepad++?

Please do a feature request here:
https://github.com/notepad-plus-plus/notepad-plus-plus/issues

I believe it’s interesting to @Scott-Sumner for implementing it ;)

Michael Vincent

@donho said in Notepad++ 7.9 RC:

Fixed and committed:
https://github.com/notepad-plus-plus/notepad-plus-plus/commit/f3fce2de4a5bf17143af9df31e67f4162d215f70

Just tested - indeed fixed, I hear no bell anymore! Thanks @donho and @scott-sumner for this request. Eagerly awaiting the full keyboard experience for Function List if #8665 gets implemented. I tried, but no luck :-(

Looking forward to 7.9 official - although I"m using the RC now happily!

Cheers.

Alan Kilborn

@deus-ex said

sort lines case insensitive

@donho There is already an open issue on it, from 2018: https://github.com/notepad-plus-plus/notepad-plus-plus/issues/4479