Skip to content

fixed Processus/FileBrowser/ReadDirectoryChanges compilation errorrs in VS2015#1472

Closed
stanciuadrian wants to merge 1 commit intonotepad-plus-plus:masterfrom
stanciuadrian:master
Closed

fixed Processus/FileBrowser/ReadDirectoryChanges compilation errorrs in VS2015#1472
stanciuadrian wants to merge 1 commit intonotepad-plus-plus:masterfrom
stanciuadrian:master

Conversation

@stanciuadrian
Copy link
Copy Markdown
Contributor

VS2015 project does not know about the file browser control. Include paths/CPP/H/RC for Processus/FileBrowser/ReadDirectoryChanges have been added/corrected. All targets compile with success: Debug/Release for x86/x64.

@Aboisier
Copy link
Copy Markdown

Your fix works, it removed VS2015 compilation errors. But I now get "ScintillaEditView::init : SCINTILLA ERROR - Can not load the dynamic library" exception. Do you know how I can fix this? Thank you!

@stanciuadrian
Copy link
Copy Markdown
Contributor Author

I think you may be mixing 32-bit and 64-bit compilers. You compile the Scintilla DLL on 32-bit and then you load it into a 64-bit process. Could you try selecting the x86 item from the Solution Platforms combo-box in the Standard toolbar from VS2015?

You can read about how to enable the 64-bit toolchain here: [https://msdn.microsoft.com/en-us/library/x4d2c09s.aspx?f=255&MSPPError=-2147217396]

You can use dumpbin /headers <dll> to find out if the dll is built for 32/64 bit.

Edited: wrong default info

@Aboisier
Copy link
Copy Markdown

I still get the same error... I realised I did not even compile Scintilla. I followed the Readme and compiled it without BOOST. I then tried to compile the VS2015 project in PowerEditor, but I still get the same error, x64 or x86.

@stanciuadrian
Copy link
Copy Markdown
Contributor Author

Did you copy SciLexer.dll to the correct location after building N++ with VS?

@Aboisier
Copy link
Copy Markdown

I most certainly did.... not! That fixed it.

Thank you very much! 👍 :)

@stanciuadrian
Copy link
Copy Markdown
Contributor Author

You're welcome! Thanks for testing my patch.

@Ede123
Copy link
Copy Markdown
Contributor

Ede123 commented Mar 1, 2016

Merging #1525 would be even better since it makes separate updates unnecessary in future.

@donho donho added the accepted label Mar 1, 2016
@donho donho closed this in 6c70948 Mar 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants