Re: Caps and Options update

[Iago Toral <itoral igalia com>]

On Tue, 05 Apr 2011 14:35:11 +0200, Guillaume Emont <gemont igalia com> 
wrote:
> Thanks for all your comments, I have pushed a new version on 
> gitorious
> that takes them in account.
>
> Here is how I see the next steps, with the aim of integration into 
> trunk:
>  0. Fix issues you guys might raise on the latest version
>  1. Remove pagination and flags from caps.
>  2. (optional) make _set_value() and _get_value() private for both
> options and caps.

Yes, I would keep them private from now and make them public once we 
find someone actually needed them.

>  3. Modify code in metadata source and media source to use use caps 
> and
> options.
>  4. Modify at least one plugin to make sure the model works
>  5. Modify all plugins to handle the new model
>  6. Test everything
>  7. (optional) merge all changes, as from this point on, we should
> normally not need to break API/ABI to add filtering and sorting 
> support

I think it would make sense.

>  8. and 9. Add filtering and sorting support, not sure what order 
> makes
> more sense. Filtering would probably be faster to implement since 
> Juan
> has already spent quite some time studying the issue.

Start with filtering then.

> Any comments on that? (especially the optional steps). Does that look
> like a sensible way of doing that?

Looks reasonable to me.

I'd add another point, to be done after all the others:

10. https://bugzilla.gnome.org/show_bug.cgi?id=645525
Add an option to let users control the priority used for idle 
operations.

Iago