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

m1 bindings and members scattered around #688

Closed
wants to merge 43 commits into from

Conversation

MarioMastr
Copy link
Contributor

this is a big one so i'll be perfectly fine with changing anything that looks wrong that i may have mised

@altalk23
Copy link
Member

altalk23 commented Aug 8, 2024

what are the 0x0 pads they are useless girl

@MarioMastr
Copy link
Contributor Author

MarioMastr commented Aug 8, 2024

wait a second those commits were added to the pr? LOL mb i didnt think they would be
ignore those

GEODE_MEMBER_CHECK(EnterEffectInstance, m_gameObject, 0x18);
GEODE_MEMBER_CHECK(EnterEffectInstance, m_unkVecInt, 0x20);

GEODE_MEMBER_CHECK(CountTriggerAction, m_unkVecInt, 0x0);
Copy link
Member

Choose a reason for hiding this comment

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

what in the world is this blasphemy

SFXTriggerGameObject* m_sfxTriggerGameObject;
}

class EventTriggerInstance {
PAD = win 0x10;
PAD = win 0x10, m1 0x0;
Copy link
Member

Choose a reason for hiding this comment

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

yeah please do not use pad 0x0 anywhere that just defeats the purpose and it doesnt even work

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the thing is if i dont put them there the members are privated

Copy link
Member

Choose a reason for hiding this comment

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

then put in the correct pads

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that's the problem, i can't see any

@altalk23 altalk23 force-pushed the main branch 2 times, most recently from b719729 to ed2b709 Compare August 8, 2024 12:00
@MarioMastr
Copy link
Contributor Author

you know what i think im just going to close this one for now

@MarioMastr MarioMastr closed this Aug 10, 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.

3 participants