FIX: [gles] do not force vsync off if VSYNC_DRIVER #4276

Merged
merged 1 commit into from Feb 27, 2014

7 participants

@koying
Team Kodi member

This solves the bug that we are forcing a vsync mode in "Driver" mode (actually we are disabling vsync as m_iVSyncMode stays at 0, then).

Generally speaking, I wonder if we need this at all, as we are apparently setting the vsync for each and every frame in Application.cpp:2239

/cc @davilla @t-nelson

@fritsch
Team Kodi member

Psst - Application.cpp is one of the sore points in xbmc, don't point the finger on it ;-)

@MartijnKaijser MartijnKaijser added this to the Pending for inclusion milestone Feb 27, 2014
@koying
Team Kodi member

Would you mind having a look at this one before beta, please.
I hurts on, e.g. imx6, where we get 80fps on framebuffer ;)

/cc @jmarshallnz

@davilla

ok

@t-nelson
@elupus
Team Kodi member
@jmarshallnz jmarshallnz merged commit f2646d9 into xbmc:master Feb 27, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment