[Bug 28018] Bind Account.ChangingPresence into Qt

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=28018

--- Comment #3 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2010-05-07 07:37:49 PDT ---
(In reply to comment #2)
> Hmm I am not sure about this, it surely would be more clear, but having one
> signal I believe it's easier to implement. We can have on signal handler to
> start/stop blinking the UI for example. Not sure here

Yeah, I'm not sure about this either - normally I'd vote for a smaller and more
consistent API (i.e. one signal with a boolean argument, like you already
implemented), but it does lead to confusingly similar method and signal names
here, so the question is whether an larger and less consistent API is a
worthwhile price for better clarity.

Keep it in the back of your mind for now, and we can make a final decision
after the spec gets released?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.