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

Aligned Scintilla and build SDK to the latest Notepad++ #5

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Bajron
Copy link

@Bajron Bajron commented May 21, 2022

Hi @KubaDee,

I like the plugin for log viewing, unfortunately I have instant crash with the latest versions of Notepad++

Tried to debug it, looked like weird stuff with pointer conversions. After reviewing the code I figured it might be the scintilla interface misalignment. I've copied scintilla/include from the Notepad++ master, and recompiled the plugin. Worked nicely for me.
I've also set the build defaults to VS2019, v142. I hope it's fine. It seems that's how the main application is built.

Would be great if we can have it released so I can have it through the plugin manager :)

Cheers,
Błażej

@sicklittlemonkey
Copy link

@Bajron

Would be great if we can have it released so I can have it through the plugin manager :)

I agree! This great plugin is one of the main reasons I use Notepad++, so thanks for your fix!

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

Successfully merging this pull request may close these issues.

2 participants