Skip to content

[Merged by Bors] - fix: prevent freeze when shoving bodies out of tiles.#637

Closed
zicklag wants to merge 1 commit intofishfolk:mainfrom
zicklag:fix/physics-freeze
Closed

[Merged by Bors] - fix: prevent freeze when shoving bodies out of tiles.#637
zicklag wants to merge 1 commit intofishfolk:mainfrom
zicklag:fix/physics-freeze

Conversation

@zicklag
Copy link
Member

@zicklag zicklag commented Feb 17, 2023

There was a scenario where a physics body could
cause an infinite loop if it ended up perfectly
aligned with the side of a tile. This prevents that
situation and panics if it ever happens again somehow.

@zicklag
Copy link
Member Author

zicklag commented Feb 17, 2023

bors merge

bors bot pushed a commit that referenced this pull request Feb 17, 2023
There was a scenario where a physics body could
cause an infinite loop if it ended up perfectly
aligned with the side of a tile. This prevents that
situation and panics if it ever happens again somehow.
There was a scenario where a physics body could
cause an infinite loop if it ended up perfectly aligned with
the side of a tile. This prevents that situation and panics if it
ever happens again somehow.
@bors
Copy link
Contributor

bors bot commented Feb 17, 2023

Build failed:

@zicklag
Copy link
Member Author

zicklag commented Feb 17, 2023

bors retry

bors bot pushed a commit that referenced this pull request Feb 17, 2023
There was a scenario where a physics body could
cause an infinite loop if it ended up perfectly
aligned with the side of a tile. This prevents that
situation and panics if it ever happens again somehow.
@bors
Copy link
Contributor

bors bot commented Feb 17, 2023

Pull request successfully merged into main.

Build succeeded:

@bors bors bot changed the title fix: prevent freeze when shoving bodies out of tiles. [Merged by Bors] - fix: prevent freeze when shoving bodies out of tiles. Feb 17, 2023
@bors bors bot closed this Feb 17, 2023
@zicklag zicklag deleted the fix/physics-freeze branch February 17, 2023 21:08
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.

1 participant