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

Fix undefined behaviour #25

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

DelilahHoare
Copy link

Fixes #23.

This prevents compiler warnings and segfaults when compiled with
recent GCC.
processevent() expects this.  This prevents the game from hanging
on input when compiled with recent Clang.
Copy link

@ljn0099 ljn0099 left a comment

Choose a reason for hiding this comment

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

This helped me to run the game.

ljn0099 pushed a commit to ljn0099/ASCIIpOrtal that referenced this pull request Dec 18, 2022
@gissf1 gissf1 mentioned this pull request Apr 18, 2024
@Apaczer
Copy link

Apaczer commented Oct 24, 2024

@zorun, maybe you could merge? In newer GCC non-void functions needs return

@PowerUser64
Copy link

Since this repository isn't getting updated (last update was 5 years ago), whoever wants this pull should just fork and merge the pull themselves, or directly use the repo that the pull is on (https://github.com/DelilahHoare/ASCIIpOrtal/tree/master). After all, that's why we have open-source. If a maintainer is done with a project or takes a break, the community can continue the project without their intervention. Everyone gets what they want.

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.

segfault at startup
4 participants