On Tue Dec 5 19:35:28 2023 +0000, Piotr Caban wrote:
This generally looks good for me. How about changing `CrtImplementationDetails_ThrowModuleLoadException_Inner` function name? I'm not familiar with the function so it's hard to suggest a better name (maybe `CrtImplementationDetails_ThrowModuleLoadException_inner` or `CrtImplementationDetails_ThrowModuleLoadException_exc`). In other functions we're using lowercase suffix if needed. This MR is marked as draft. Please mark it as ready when appropriate.
I will rename to CrtImplementationDetails_ThrowModuleLoadException_inner and What I found that from extensive searching on google and the error messages is that the it trys to rethrow the inner exception as ModuleLoadException. eg: https://forum.unity.com/threads/c-wrapper-for-unity.265398/