On Tue, Nov 20, 2012 at 07:53:46PM +0200, Zeeshan Ali (Khattak) wrote: > > To have a friendly external API rather than forcing internal implementation > > issues onto the library user. > > Thinking more about this, other restrictions could theoretically also > be >1. Since I'm not sure we'll actually be encountering such > installers, I think we currently go with this API Not sure what you call this API, but my gut feeling is that installers that support a single image format are the exception rather than the norm. > and when we actually > encounter such an installer, we: > > 1. Allow multiple avatar formats, i-e add a > osinfo_install_script_get_avatar_formats() > 2. Make _get_avatar_format() return the first restriction in the list > 3. Document in _get_avatar_format() about #2 and that apps should use > the new API Could get a bit ugly with respect to gobject properties, so better if we can avoid going down that road ;) Also, I think we are more likely to have installers needing ((PNG or JPEG) and (64x64 or 128x128)) rather than ((PNG and 64x64) or (JPEG and 128x128)). The API you suggest seems appropriate in the latter case, not the former. Christophe
Attachment:
pgpoKfK1TaGjh.pgp
Description: PGP signature
