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

Fixed Issue #209: Walking through blocks at a 45 degree angle #216

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

Conversation

usernamefreaks
Copy link

Link to the issue we fixed: #209

The cause of the bug is that in the collide() function, it doesn't check whether the 4 diagonally neighboring blocks are obstacles. We added code that check for obstacles in these directions.

pspiworld added a commit to pspiworld/piworld that referenced this pull request Oct 25, 2019
Fixed in Craft pull request fogleman#216
from usernamefreaks.

Bug report: fogleman#209
Copy link

@ihalseide ihalseide left a comment

Choose a reason for hiding this comment

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

The code seems good. Just has some minor formatting issues: mixing tabs and spaces, and there are distracting comments.

Copy link

@ihalseide ihalseide left a comment

Choose a reason for hiding this comment

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

I think this pull request is great now. It is a good improvement to the game.

tsee pushed a commit to tsee/Craft that referenced this pull request May 29, 2023
Fixed in Craft pull request fogleman#216
from usernamefreaks.

Bug report: fogleman#209
tsee pushed a commit to tsee/Craft that referenced this pull request Jun 2, 2023
Fixed in Craft pull request fogleman#216
from usernamefreaks.

Bug report: fogleman#209
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.

Collision issues (walking through blocks)
4 participants