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

Action Bools go bye bye, hello Action Enum #239

Open
wants to merge 89 commits into
base: nightly
Choose a base branch
from

Conversation

Amy54Desu
Copy link
Contributor

Basically what I'm trying to do is get rid of ALL action booleans and uses a new action system to handle things like Propeller Spinning and stuff.

This starts with the action declaration, flags, and a few adjustments. Currently only done for hip drops
We weren't using the .qtn files, now we do
Update this to use action flags
This removes the boolean for IsDrilling and IsGroundpounding and replaces them with the setPlayerAction
This adds in 2 new flags and a few missing actions
Replaces the skidding boolean with the player action
This is part one, I'll do part two later
This adds a new method called checkActionFlags that should make it less convoluted
We now have the hasActionFlags method, use it over (mario->actionFlags & <actionFlag>) != 0
This adds in a Pipe action, and makes the Bobomb use the IsShelled action flag
the old system used a bool called "weak"
This adds in a new flag called "Camera Change" as well as fixing problems
SetPlayerAction now uses Frame f, and Knockback now uses the actionArg for how many stars to drop and the direction
let's make the frame be optional
Changes all knockback stuff to use our SetPlayerAction
these are new methods for MarioPlayer
This implements the "Check Entity Bounce" method
Mini Ground Pound has been added. It is like the regular one except it doesn't have the NoPlayerBounce flag
You heard me right, a new system for handling actions
Main ones being SetPlayerAction now making the Frame MANDATORY, PreActionInput and the Bounce action now being added
we're getting rid of ->DoEntityBounce and replacing it with an action
now uses PlayerAction.Bounce
... and... adjustments to JumpHandler
This is the beginning of the hold actions
@Windows10V1
Copy link

holy shit 75 coommits

@ArianLust
Copy link

Joe Ben

@Windows10V1
Copy link

pfft, that sucked

Wallkicks now work as intended
Adds a new method called EnableSpinner as well as a few fixes
This commit focuses on the death aspect, removing the player delete from Respawn and putting it into the Death Action instead. It also now speeds up the star drop rate for every 10 stars that drop.
Just a change to speeding up every 5 stars
Fix the hip drop check so it properly functions, and do a bit of fixing of the Blue Shell
I'm Isaac Newton, mainly physics stuff
Removes the BreaksBlocks and Strong Action flags in favor of the new StompPowerLevel int. A bunch of mini mushroom checks are also removed for the StompPowerLevel.
First off, I don't know what I was thinking, why the hell did I make StompPower an int instead of an enum smh smh.
Second off, starting to make water actions also an enum
We now have water actions! Let's use 'em!
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