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
Consider using Native Library Loading #226
Comments
I'm a bit confused on the benefit of this. In addition to the large scale rewrite of P/Invoke definitions, wouldn't this also either require users to manually call some static method we create like What do we gain out of this change? |
I'm wrong. I think the main confusing with all of this is, that if the native lib is found, but then fails to load due to a missing dependency, DllImport will simply continue loading other libraries. The messaging around this is not very clear, so at a glance it appears as if the specific library was never considered, while it was actually always a dependency issue. This now also allowed me to revert the change made regarding extension fixing in CSFML. |
I don't necessarily think Native Library Loading is needed and may actually not make sense. If we need to influence the loading behavior we may probably first want to write our own Custom Import Resolver. As such, I'll close this issue for now. |
Using p/invoke with
DllImportAttribute
is a bit cumbersome in a cross-platform manner.As such, it might make sense to use the NativeLibrary class instead.
References
The text was updated successfully, but these errors were encountered: