Skip to content

Small fixes#94

Merged
ManlyMarco merged 7 commits intoBepInEx:masterfrom
limoka:small-fixes
Jul 26, 2023
Merged

Small fixes#94
ManlyMarco merged 7 commits intoBepInEx:masterfrom
limoka:small-fixes

Conversation

@limoka
Copy link
Copy Markdown
Contributor

@limoka limoka commented Jun 10, 2023

This PR combines a number of small fixes:

  1. Fix inherited injected fields not being initialized on derived classes
  2. Fix errors injecting a type inheriting a abstract type. I'm not entirely sure why this happens, but the baseVTablePointer[i].method in my case was not default, even though all of it's fields were null.
  3. Fix delegate type names including assembly names. This might cause very long names and if such name becomes too long (I think 1024 characters or more), the runtime will not like it.
  4. Fix NRE setting null to Value of Il2CppReferenceField.
  5. Fix Il2Cpp Detour not being applied if target type is a struct. I have tested this change and it does not cause any issues. Patches get called correctly, and ref MyStruct __instance injection works correctly.

I have tested these changes on Core Keeper Unity 2021.3.14.

Co-authored-by: Kasuromi <29818914+Kasuromi@users.noreply.github.com>
@limoka limoka requested a review from Kasuromi June 10, 2023 10:25
@ManlyMarco ManlyMarco merged commit f3533f9 into BepInEx:master Jul 26, 2023
@ds5678 ds5678 added this to the 1.4.6 milestone Jul 15, 2024
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.

4 participants