Tone: Performance improvements #1017

merged 6 commits into from Sep 1, 2016


None yet

2 participants

Ghabry commented Aug 28, 2016

Tone change of the screen is now calculated per sprite. This means the tone is only calculated once and afterwards the Blit is normal speed as if no tone is changed. Special case is a "gradient change", when the tone changes per frame. In that case it uses the old behaviour because is faster for that case.

Gives a significant speed boost on slow hardware for that case obviously ;)

Ghabry added some commits Aug 26, 2016
@Ghabry Ghabry Improve performance of Tone effect by only rendernig the tone effect …
…once per sprite instead of applying it to the screen. Respect x and y for the tone change.
@Ghabry Ghabry Tilemap: Prevent cache collisions 276613d
@Ghabry Ghabry Plane: Add Tone Blit 99effa7
@Ghabry Ghabry Spriteset_Map: Switch from stack to heap allocation 58272cd
@Ghabry Ghabry Move Map/Battle sprites to SpriteSet class. Use screen tone for gradi…
…ent changes, use per-sprite tone for constant tone (faster for these situations).
@Ghabry Ghabry Weather is below screen and needs tone change, too.
Ghabry commented Aug 29, 2016

This could have regressions, though unlikely. Will also ask Wii people to test ;).

@fdelapena fdelapena added this to the 0.5.0 milestone Aug 29, 2016
Ghabry commented Aug 30, 2016

Got feedback in the forum, they are happy ;)

@fdelapena fdelapena merged commit 7564390 into EasyRPG:master Sep 1, 2016

6 checks passed

Android (armeabi-v7a) Build finished.
GNU/Linux Build finished.
OSX Build finished.
Windows (x64) Build finished.
Windows (x86) Build finished.
web Build finished.
@Ghabry Ghabry deleted the Ghabry:cache_tone branch Sep 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment