xbmc/xbmc — pull request #1006 by jmarshallnz (2012-05-24 06:39:54+00:00)
Title: Return appropriate type from Addon.getSetting()

This is nowhere near acceptable (essentially hides the ickyness in CAddon), but just wanted to get it out there given there's talk of non-backward compatible changes to the python API. To do it properly we need to decide on how we wish to store settings within CAddon - keeping XML doesn't seem to be appropriate - not sure whether we want something similar to GUISettings (i.e. base settings class with derived classes for different settings types) or not. Comments welcome.

What currently isn't handled is verification of the values set by python (eg they could set "bob" as the value of a bool setting if they wanted to). It might pay, for example, to allow setSetting() to take different types (bool, int, float, string).

@jimfcarroll: this may well have implications for your stuff.