search

Interrupt / systemshock

Shockolate - A minimalist and cross platform System Shock source port.
GNU General Public License v3.0
811 stars 66 forks source link

Fix divide by zero crash #170

Closed donnierussellii closed 6 years ago

donnierussellii commented 6 years ago

Logic of function did not check specifically for a divide by zero. Multiple crashes on level 3, probably elsewhere.

Lots of playtesting seems to confirm it's fixed now.

Interrupt commented 6 years ago

This PR also includes all of the commits from the relative mouse movement PR, although I see that you rolled them back. In the future to make things cleaner, you could make a new branch off of Interrupt/master and cherry-pick your one commit onto that.

donnierussellii commented 6 years ago

I made commits to master, then made branches that inherited those commits. Once it's done all I can do is recommit it back to the way it was. I wish I could just start over and do it right. Learning the hard way....