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 tileLoadTile to check if tile already loaded #75

Merged
merged 1 commit into from Jun 2, 2019

Conversation

Projects
None yet
3 participants
@Phlamethrower
Copy link
Contributor

commented Jun 2, 2019

While working on my RISC OS port I spotted that there was a lot of stuttering, especially when firing weapons. Some debugging revealed that it was because tileLoadTile wasn't checking to see if the tile was already loaded before calling tileLoad.

@nukeykt

This comment has been minimized.

Copy link
Owner

commented Jun 2, 2019

Merging, thanks

@nukeykt nukeykt merged commit 38d957d into nukeykt:master Jun 2, 2019

3 checks passed

Travis CI - Pull Request Build Passed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Rinnegatamante

This comment has been minimized.

Copy link

commented Jun 5, 2019

@Phlamethrower related to this issue. After your PR, do you have huge stuttering issues when the second demo pops in after the first one finished in main menu? Your PR actually solved stuttering issues in my PSVITA port too in game however that second demo completely kills framerate on Vita (getting like 1 new frame every 3-4 seconds).

@Phlamethrower

This comment has been minimized.

Copy link
Contributor Author

commented Jun 5, 2019

Yeah, that level (e3m2) does seem pretty laggy. I'll see if I can take a closer look next time I do some work on the project - hopefully sometime in the next week or two.

@Phlamethrower Phlamethrower deleted the Phlamethrower:fix-tileload branch Jun 5, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.