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

Slightly better way to toggle nogc #27

Merged
merged 1 commit into from Aug 4, 2018

Conversation

Projects
None yet
2 participants
@fgsfdsfgs
Contributor

fgsfdsfgs commented Aug 4, 2018

I put patchFilter[] in there for a reason, you know :^)
Also I think it's better to leave it enabled by default, just in case someone forgets or doesn't know about the nogc file, since that could cause irreversible changes.

@Reisyukaku

This comment has been minimized.

Show comment
Hide comment
@Reisyukaku

Reisyukaku Aug 4, 2018

Owner

That whole kippatch codebase triggers me and needs to be rewritten.. so I made the hacky patch for now. but yea that works lol

Owner

Reisyukaku commented Aug 4, 2018

That whole kippatch codebase triggers me and needs to be rewritten.. so I made the hacky patch for now. but yea that works lol

@Reisyukaku Reisyukaku merged commit 1235dfa into Reisyukaku:master Aug 4, 2018

@fgsfdsfgs

This comment has been minimized.

Show comment
Hide comment
@fgsfdsfgs

fgsfdsfgs Aug 4, 2018

Contributor

Understandable. Note that this shit changes the nogc "mechanics" and you now need to create a disable_nogc file to disable nogc. Don't even know if that's fine or you just didn't notice.

Contributor

fgsfdsfgs commented Aug 4, 2018

Understandable. Note that this shit changes the nogc "mechanics" and you now need to create a disable_nogc file to disable nogc. Don't even know if that's fine or you just didn't notice.

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