Skip to content
This repository has been archived by the owner on Jun 5, 2023. It is now read-only.

Implemented Dispose #18

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Conversation

ajmcateer
Copy link

I implemented dispose in the WebWindow but I am having trouble getting it to work in Windows.
Linux was a complete shot in the dark as I have never worked with C++ let alone on Linux before.
I get

Exception thrown: 'System.EntryPointNotFoundException' in WebWindow.dll

whenever I try to call the dispose method. As far as I can tell I implemented everything correctly, but I am knew to C++ so its very likely I missed something. Any feedback would be greatly appreciated. I also didn't see a WebWindow.Mac.cpp file to make changes too.

@Berrysoft
Copy link
Contributor

I also didn't see a WebWindow.Mac.cpp file to make changes too.

It's WebWindow.Mac.mm.

@SteveSandersonMS
Copy link
Owner

@ajmcateer Thanks for contributing this!

Do you think you'll be in a position to finish this PR off? That would involve figuring out the solution to whatever issue you're describing, as well as tidy this up to remove the unnecessary changes (e.g., in WebWindow.Native.vcxproj).

If that's not something you'll get round to, no problem - I may add some functionality like this myself eventually, though there's no ETA.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants