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

Add fully quality module name and remove the android handled #531

Merged
merged 2 commits into from
May 27, 2024

Conversation

xenolightning
Copy link
Contributor

Fix the module name, to fully qualify the path to load.

That way if the executing directory is different to the module directory, it can load the file correctly.

Also remove the unhandled = true flag from Android

@@ -49,7 +49,6 @@ private static void AndroidUnhandledExceptionHandler(object sender, object excep
{
var exceptionArgsType = exceptionArgs.GetType();
exception = (Exception)exceptionArgsType.GetProperty("Exception").GetValue(exceptionArgs);
exceptionArgsType.GetProperty("Handled")?.SetValue(exceptionArgs, true);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't actually need this, this just prevents the app from crashing on android.

If this is desired behaviour the user can hook this and do it in their code, but we shouldn't be affecting functionality

Copy link
Contributor

@ProRedCat ProRedCat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@xenolightning
Copy link
Contributor Author

Released as 11.0.0-pre2

@xenolightning xenolightning merged commit 6678ade into master May 27, 2024
1 check passed
@xenolightning xenolightning deleted the sean/portable-pdb-extensions branch May 27, 2024 03:28
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