xbmc/xbmc — pull request #1830 by Montellese (2012-11-22 18:41:51+00:00)
Title: settings: update videoscreen.screenmode value when updating to Frodo

Found another setting that doesn't update properly from Eden to Frodo: videoscreen.screenmode. As it is used for screen and resolution selection we should try to update the old (Eden) value. All that has changed in Frodo is that we add a "p" or an "i" at the end to indicate progressive or interlaced. So it's pretty easy to detect the update because the length of the value of videoscreen.screenmode goes from 20 to 21. This simply adds a "p" at the end to make the value compatible with Frodo (otherwise it completely falls back to RES_DESKTOP). Another option would be to change the implementation of CGUISettings::GetResFromString() to be able to handle values with or without the progressive/interlaced indicator.

Let me know which approach you prefer. I don't think there's a way to actually make an educated guess about progressive or interlaced at the time we load the GUI settings because the windowing system is initialized later so we can't just use whatever e.g. RES_DESKTOP uses.