Skip to content

Player: Implement YoshiStateEgg - #1183

Open
guymakinggames wants to merge 1 commit into
MonsterDruide1:masterfrom
guymakinggames:YoshiStateEgg
Open

Player: Implement YoshiStateEgg#1183
guymakinggames wants to merge 1 commit into
MonsterDruide1:masterfrom
guymakinggames:YoshiStateEgg

Conversation

@guymakinggames

@guymakinggames guymakinggames commented May 5, 2026

Copy link
Copy Markdown
Contributor

This change is Reviewable


Report for 1.0 (c4d724d - 7f959c5)

📈 Matched code: 15.14% (+0.01%, +1124 bytes)

✅ 10 new matches
Unit Item Bytes Before After
Player/YoshiStateEgg (anonymous namespace)::updateEggMove(al::LiveActor*, IUsePlayerCollision const*, PlayerConst const*) +260 0.00% 100.00%
Player/YoshiStateEgg YoshiStateEgg::YoshiStateEgg(al::ActorInitInfo const&, al::LiveActor*, IUsePlayerCollision const*, PlayerConst const*, al::WaterSurfaceFinder const*, PlayerModelChangerYoshi*) +188 0.00% 100.00%
Player/YoshiStateEgg YoshiStateEgg::reactionCollidedCollisionCode() +128 0.00% 100.00%
Player/YoshiStateEgg YoshiStateEgg::appear() +104 0.00% 100.00%
Player/YoshiStateEgg (anonymous namespace)::YoshiStateEggNrvWait::execute(al::NerveKeeper*) const +104 0.00% 100.00%
Player/YoshiStateEgg YoshiStateEgg::exeWait() +100 0.00% 100.00%
Player/YoshiStateEgg (anonymous namespace)::YoshiStateEggNrvAppear::execute(al::NerveKeeper*) const +96 0.00% 100.00%
Player/YoshiStateEgg YoshiStateEgg::exeAppear() +92 0.00% 100.00%
Player/YoshiStateEgg YoshiStateEgg::~YoshiStateEgg() +36 0.00% 100.00%
Player/YoshiStateEgg YoshiStateEgg::kill() +16 0.00% 100.00%

@github-actions github-actions Bot added the status:waiting for review This PR is ready to take the first or another look at it! label May 5, 2026
@MonsterDruide1 MonsterDruide1 added the ai This PR has been created with the help of AI tools and is handled with a lower priority. label May 8, 2026

@german77 german77 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@german77 made 1 comment.
Reviewable status: 0 of 5 files reviewed, 1 unresolved discussion (waiting on guymakinggames).


src/Player/YoshiStateEgg.cpp line 84 at r1 (raw file):

                               gravityValue * gravity.z};
    al::tryAddVelocityLimit(actor, velocity, playerConst->getFallSpeedMax());
}

Suggestion:

    al::tryAddVelocityLimit(actor,  al::getGravity(actor)*playerConst->getGravity(), playerConst->getFallSpeedMax());
}

@github-actions github-actions Bot added status:waiting for author Reviews have shown that the author of this PR needs to change some things. and removed status:waiting for review This PR is ready to take the first or another look at it! labels May 10, 2026

@guymakinggames guymakinggames left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@guymakinggames made 1 comment.
Reviewable status: 0 of 5 files reviewed, 1 unresolved discussion (waiting on german77).


src/Player/YoshiStateEgg.cpp line 84 at r1 (raw file):

                               gravityValue * gravity.z};
    al::tryAddVelocityLimit(actor, velocity, playerConst->getFallSpeedMax());
}

Done.

@github-actions github-actions Bot added status:waiting for review This PR is ready to take the first or another look at it! and removed status:waiting for author Reviews have shown that the author of this PR needs to change some things. labels May 22, 2026
@german77

german77 commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Superseded by #1333

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

Labels

ai This PR has been created with the help of AI tools and is handled with a lower priority. status:waiting for review This PR is ready to take the first or another look at it!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants