Skip to content

Conversation

@Banane9
Copy link
Contributor

@Banane9 Banane9 commented Jun 2, 2025

Adds a custom ResoniteBugFixMonkey base type for the fixes that handles some basics like making the BugFixOptions available in all of them

@Banane9 Banane9 added the enhancement New feature or request label Jun 2, 2025
@Banane9 Banane9 merged commit 7864172 into master Jun 2, 2025
1 check passed
@Banane9 Banane9 deleted the refactor/bugfixmonkey branch June 2, 2025 22:53
[HarmonyPatchCategory(nameof(ValueModDecimal))]
[HarmonyPatch(typeof(Elements.Core.Coder<Decimal>), nameof(Elements.Core.Coder<Decimal>.Mod))]
internal sealed class ValueModDecimal : ResoniteMonkey<ValueModDecimal>
[HarmonyPatch(typeof(Coder<decimal>), nameof(Coder<Decimal>.Mod))]
Copy link
Contributor

Choose a reason for hiding this comment

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

different casing for decimal vs Decimal, not sure if this causes issues?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

decimal is just a C# alias for System.Decimal - I just changed it for consistency, not sure why it got in differently 😅

Copy link
Contributor Author

Choose a reason for hiding this comment

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

And oh, I see what you mean now, one is still capitalized... Huh

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it should be fine, in the decompiler they both point to the same type

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yea, not worried about that, I just want it consistent 😂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants