zeitgeist team mailing list archive

[Manish Sinha <612344@xxxxxxxxxxxxxxxxxx>]

> 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
https://bugs.launchpad.net/bugs/612344
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...