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

Remove config define bloat #436

Open
Reonu opened this issue Jun 29, 2022 · 12 comments
Open

Remove config define bloat #436

Reonu opened this issue Jun 29, 2022 · 12 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed high priority Important, non-critical issue or feature / high priority subjective Multiple opinions are desirable
Milestone

Comments

@Reonu
Copy link
Contributor

Reonu commented Jun 29, 2022

SIDE_FLIP_AT_LOW_SPEEDS this seems pointless

NO_FALL_DAMAGE and NO_FALL_DAMAGE_SOUND I think they should just be combined into a single one (what's the point of the scream if it doesn't actually warn you of fall damage)

FALL_DAMAGE_HEIGHT_SMALL and FALL_DAMAGE_HEIGHT_LARGE are a bit bloaty maybe

SHINDOU_POLES is this worth a define?

EASIER_LONG_JUMPS DISABLE_GROUNDPOUND_BONK JUMP_KICK_FIX HANGING_FIX remove the defines and force them on

Feel free to discuss

EDIT:
CURRENT PLANS: https://docs.google.com/spreadsheets/d/1j2iStJFkhjGYap-t8HS9Jb2uopGK1HSUclOb2vQqzEU/edit?usp=sharing

@Reonu Reonu added enhancement New feature or request help wanted Extra attention is needed questionable This is risky or potentially invalid or undesirable labels Jun 29, 2022
@Reonu Reonu added this to the 2.1 milestone Jun 29, 2022
@Reonu Reonu self-assigned this Jun 29, 2022
@thecozies
Copy link
Contributor

EASIER_LONG_JUMPS could be default. no reason to need a frame one ground pound. just put a comment explaining the fix

@thecozies
Copy link
Contributor

DISABLE_GROUNDPOUND_BONK
imo enabled by default. its only painful and i dont think any other mario games have done this since

@Reonu
Copy link
Contributor Author

Reonu commented Jun 29, 2022

I agree with both

@Reonu Reonu closed this as completed Jun 29, 2022
@Reonu Reonu reopened this Jun 29, 2022
@Reonu
Copy link
Contributor Author

Reonu commented Jun 29, 2022

I did not mean to close that

@Reonu Reonu closed this as completed Jun 29, 2022
@Reonu Reonu reopened this Jun 29, 2022
@Reonu
Copy link
Contributor Author

Reonu commented Jun 29, 2022

Why does github default to closing the issue when pressing ctrl+enter in the comment box

Fuck you

@thecozies
Copy link
Contributor

// Allows Mario to jump kick on steep surfaces that are set to be non slippery, instead of being forced to dive.
#define JUMP_KICK_FIX

maybe just enabled? this is a weird one

@Reonu
Copy link
Contributor Author

Reonu commented Jun 29, 2022

HANGING_FIX this should be removed as well, and force it on

with it disabled you can only grab ceilings from a single or double jump, why would you ever want that?

@thecozies
Copy link
Contributor

DISABLE_BLJ you can search the code for "blj" and find exactly where the cause is already. most people wont disable this and if somebody really wants to AND cant figure out how, its a one line change that somebody could refer them to.

this also brings up an idea of adding comments for things and encourage people to use the search feature and read the code which would help them learn instead of having a magic toggle and not knowing what it actually does. this isn't assembly, if a noob can read it and make the change, then we dont need a define

@gheskett
Copy link
Collaborator

NO_FALL_DAMAGE_SOUND as a define doesn't need to exist, though I personally think the falling scream should be present regardless of whether fall damaged is disabled.

@gheskett
Copy link
Collaborator

FALL_DAMAGE_HEIGHT_SMALL and FALL_DAMAGE_HEIGHT_LARGE can probably be moved out of the global config and defined elsewhere

@Arceveti Arceveti linked a pull request Jul 19, 2022 that will close this issue
@Arceveti Arceveti added waiting on pull request This issue has an existing PR which fixes it and removed waiting on pull request This issue has an existing PR which fixes it labels Jul 19, 2022
@Arceveti Arceveti removed a link to a pull request Jul 19, 2022
@Tubular-Bells
Copy link
Contributor

SHINDOU_POLES
Could we include an explanation of what the differences between normal poles and Shindou poles are? This assumes the define is kept at all, but I can't comment on whether that's a good idea because I still don't know what it does 💀

@gheskett gheskett modified the milestones: 2.1, 3.0 Mar 6, 2023
@gheskett gheskett added subjective Multiple opinions are desirable and removed questionable This is risky or potentially invalid or undesirable labels Jun 18, 2023
@gheskett gheskett changed the title Remove possibly unnecesary defines in config_movement.h (discuss) Remove config define bloat Dec 16, 2023
@gheskett gheskett added critical Extremely important issue or feature / highest priority high priority Important, non-critical issue or feature / high priority and removed critical Extremely important issue or feature / highest priority labels Dec 16, 2023
@gheskett
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed high priority Important, non-critical issue or feature / high priority subjective Multiple opinions are desirable
Projects
Status: In progress
Development

No branches or pull requests

5 participants