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

Include Slayer in "All Guns" cheat #442

Open
wants to merge 1 commit into
base: port
Choose a base branch
from

Conversation

blaiz
Copy link

@blaiz blaiz commented May 20, 2024

The "All Guns" cheat doesn't include the Slayer, probably because the devs didn't want us peeking above walls to reveal missing level geometry. Let's remedy this though, I like peeking behind level boundaries. This is a childhood dream come true. If we're gonna have a cheat to allow dual wielding all weapons, then this should be ok to do as well I assume.

Screenshot 2024-05-19 191204

Comment on lines 284 to 285
if (mainGetStageNum() != STAGE_ATTACKSHIP
&& mainGetStageNum() != STAGE_SKEDARRUINS
Copy link
Author

Choose a reason for hiding this comment

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

On the N64 there are 2 exception to the "no Slayer for All Guns" rule which is these 2 stages. Let's make it so we get the behavior for ALL levels.

@fgsfdsfgs
Copy link
Owner

fgsfdsfgs commented May 20, 2024

While it's nice to have this as an option, there's one pretty significant reason the Slayer is only available on certain maps. If the map wasn't made with flying around in mind, it's pretty easy to encounter portal bugs like this:
image
Not to mention you can also get into out of bounds areas.

Like with your other Slayer PR I'd rather like this to be a separate cheat. If the people don't want to encounter the jank, they can just use the normal All Guns.

@blaiz
Copy link
Author

blaiz commented May 20, 2024

Makes sense. I’ll see if I can make this a Slayer gun cheat specifically.

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