Notepad++ v8.8.5 Release

donho

Notepad++ release 8.8.5 is available:
https://notepad-plus-plus.org/news/v885-released/

Notepad++ release 8.8.5 virus scanner issue? Please check the following link:
https://notepad-plus-plus.org/news/v883-self-signed-certificate/

Notepad++ v8.8.5 regression-fix:

Fix “Edit with Notepad++” context menu not being installed correctly regression. (Fix #16913)

Notepad++ v8.8.4 regression-fixes, bug-fixes & new features:

Fix -notabbar & asNotepad.xml stop hiding tabbar regression (from v8.7.9). (Fix #16794)
Fix “Edit with Notepad++” not removed by uninstaller regression (from v8.8.3). (Fix #16896)
Fix possible heap memory leak. (implement #16883)
Increase search input length from 2046 to 16383. (Fix #1088, #16749)
Add warning tip for the max length on search input (Find/Replace). (Fix #16749)
Fix JavaScript listed twice in Language & Indent settings of Preferences issue. (Fix #4521, #16861, #16884)
Fix external lexers showing several time in Indent settings issue. (Fix #16462)
Add button to insert variables in the run dialog. (implement #16820)
Enhance “Go to settings”: guide users to the related setting explicitly. (Fix #16846)
Add a clear button for filter field in Shortcut Mapper. (Fix #15742)
Make NFO font overridable by adding font name in Lexer “nfo” of stylers.xml. (Fix #16839)

As usual, auto-updater will be triggered in one week, if no critical issue found.

Vitalii Dovgan

@donho
Looks like the behavior of NPPM_RELOADFILE has been changed in either 8.8.5 or 8.8.4.
Previously, NPPM_RELOADFILE activated the file being reloaded. In 8.8.5, the file is not activated.
Is it intended?

donho

@Vitalii-Dovgan

Looks like the behavior of NPPM_RELOADFILE has been changed in either 8.8.5 or 8.8.4.

I don’t think so.
Here is git blame for the API you’re talking about:

Obviously, the latest changing is the last year. So it was not changed in v8.8.4.
BTW, you’re the only one (AFAICT) who complains this problem.
If you find any issue, you’re welcome to create an issue on GitHub, and ping me - I’ll check it.

Vitalii Dovgan

@donho
Oh my God, you are right!
Looks like NPPM_RELOADFILE has actually never activated the file being reloaded.
The only change introduced in NPPM_RELOADFILE was between 8.6.4 and 8.6.5 where NPPM_RELOADFILE started to return TRUE on success. Previously it was always returning FALSE. And it was NppExec that activated the file when NPPM_RELOADFILE had returned FALSE.
And all this time I’ve been mistakenly thinking that NPPM_RELOADFILE activates the file being reloaded.
Actually, the following code in Notepad_plus::doReload seems to be related to activating the buffer, so I’m not sure what actually happens:

if ( ((currentView() == MAIN_VIEW) && mainVisible) || ((currentView() == SUB_VIEW) && subVisible))
{
	activateBuffer(id, currentView(), true);
}