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

Fill out more of EnhacnedGameObject's Class members and other missing… #860

Merged
merged 5 commits into from
Dec 25, 2024

Conversation

CallocGD
Copy link
Contributor

Hopefully this pull request is better than yesterday's one where I was up at 2AM. Seems having GameObject's previous class members helped out but I am still hunting down the remaining ones that slipped through the cracks somehow thanks to this update.

@CallocGD
Copy link
Contributor Author

I swear the verification system is screwed up.

bool m_activatedByPlayer1;
bool m_activatedByPlayer2;
bool m_hasUniqueCoin;
cocos2d::CCColor3B m_unkAnimationColor;
Copy link
Member

Choose a reason for hiding this comment

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

this is supposed to be ccColor3B not uppercase

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@CallocGD CallocGD requested a review from altalk23 November 28, 2024 01:33
Copy link
Member

@altalk23 altalk23 left a comment

Choose a reason for hiding this comment

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

(last member) I'm not so sure about the validity of this as first member of effectgameobject is also cccolor3b (and pads dont match)
member tests could be helpful

@altalk23
Copy link
Member

wow github is weird

@@ -4770,8 +4770,19 @@ class EnhancedGameObject : GameObject {
void updateUserCoin() = win 0x1a3970, imac 0x5a9770, m1 0x4e3b94;
TodoReturn waitForAnimationTrigger();

PAD = android32 0x25, win 0x2a, android64 0x22, mac 0x22;

bool m_poweredOn;
Copy link
Member

Choose a reason for hiding this comment

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

this ones a weird one because android32 becomes 0x26 but others are fine, maybe the old pad was wrong

bool m_activatedByPlayer1;
bool m_activatedByPlayer2;
bool m_hasUniqueCoin;
cocos2d::ccColor3B m_unkAnimationColor;
Copy link
Member

@altalk23 altalk23 Nov 28, 2024

Choose a reason for hiding this comment

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

I'm not so sure about the validity of this as first member of effectgameobject is also cccolor3b (and pads dont match)
member tests could be helpful

@altalk23 altalk23 merged commit 9ab5fa9 into geode-sdk:main Dec 25, 2024
13 checks passed
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