> Let 'E' define the event dbus signature.
> Methods:
>  - GetTemplates() -> ({sE})
>  - AddTemplate(s, E)
>  - RemoveTemplate(s)
> Signals:
>  - Changed({sE})
> Apps would use their normal namespaced bus names to construct blacklist 
> names. 
> Like 'org.gnome.Epiphany.PrivateBrowsing' for the blacklist template epiphany 
> install when you enter private mode.

I was looking at this bug today and wanted to clarify some points.

1) s =  'org.gnome.Epiphany.PrivateBrowsing'  ??

2) So ({sE}) means each blacklist template associated with the
application which set it

3) RemoveTemplate(s) would clear all the blacklists set by an
application. Should not it be RemoveTemplate(s, E) or RemoveTemplate(s,
{E}) meaning clearing more than one event template at a time

4) Similarly, how about AddTemplate(s, {E}) meaning adding more than one
template at a time

Have some more doubts on Changed() but will ask them after these 4
doubts are cleared.

Blacklist API sucks
You received this bug notification because you are a member of Zeitgeist
Framework Team, which is subscribed to Zeitgeist Framework.

Status in Unity: Triaged
Status in Unity Files Place: Triaged
Status in Zeitgeist Framework: Triaged

Bug description:
Guys, GetBlacklist and SetBlacklist (without any signals) for an 
asynchrounous-by-nature API? Come on!

How about changing it to Get, Add, Remove and a changed signal? That way it'd 
be actually usable...

Mailing list: https://launchpad.net/~zeitgeist
Post to     : zeitgeist@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zeitgeist
More help   : https://help.launchpad.net/ListHelp

Reply via email to