Skip to content

Fix GetIl2CppNestedType sometimes failing#227

Merged
ds5678 merged 2 commits intoBepInEx:masterfrom
slxdy:GetIl2CppNestedType-fix
Aug 23, 2025
Merged

Fix GetIl2CppNestedType sometimes failing#227
ds5678 merged 2 commits intoBepInEx:masterfrom
slxdy:GetIl2CppNestedType-fix

Conversation

@slxdy
Copy link
Contributor

@slxdy slxdy commented Aug 23, 2025

Added a fallback method for GetIl2CppNestedType.
This fixes GetIl2CppNestedType sometimes failing to find the nested type due to il2cpp somehow giving the wrong nested type names through il2cpp_class_get_nested_types and il2cpp_class_get_name. I have no clue what the exact cause is for the issue, but the fallback works just fine.

slxdy added 2 commits August 23, 2025 03:12
Added a fallback method for GetIl2CppNestedType.
This fixes GetIl2CppNestedType sometimes failing to find the nested type due to il2cpp somehow giving the wrong nested type names through `il2cpp_class_get_nested_types` and `il2cpp_class_get_name`.
I have no clue what the exact cause is for the issue, but the fallback works just fine.
@slxdy
Copy link
Contributor Author

slxdy commented Aug 23, 2025

Sorry for the extra merge. Forgot my fork was actually of Jeremy's fork which was slightly outdated

@ds5678 ds5678 merged commit 7785052 into BepInEx:master Aug 23, 2025
2 checks passed
slxdy added a commit to slxdy/Il2CppInterop that referenced this pull request Aug 23, 2025
Fixes BepInEx#220 which completely broke most of the Il2Cpp imports.
Reverts BepInEx#227. The fallback method for GetIl2CppNestedType is no longer needed as the root cause of the issue has been solved.
@slxdy slxdy mentioned this pull request Aug 23, 2025
ds5678 pushed a commit that referenced this pull request Aug 23, 2025
* Fix UTF8 marshalling for IL2CPP imports

Fixes #220 which completely broke most of the Il2Cpp imports.
Reverts #227. The fallback method for GetIl2CppNestedType is no longer needed as the root cause of the issue has been solved.

* Add UTF-8 support to the Class Injector
@ds5678 ds5678 added this to the 1.5.1 milestone Sep 2, 2025
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