[Bug 27727] account storage plugins can contain partial accounts

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Thu Nov 14 11:20:30 PST 2013


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

--- Comment #32 from Xavier Claessens <xclaesse at gmail.com> ---
(In reply to comment #14)
> Created attachment 89217 [details] [review]
> 10/26] mcd_account_delete: convert into  mcd_account_delete_async

migrate_delete_account_cb() looks weird, why don't you put directly
mcd_account_delete_debug_cb as callback and free the MigrateCtx before the
async call?

While we are adding async API, why omitting the GCancellable? I think you can
just have one and put it on your GTask, it will do the right thing for you.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.


More information about the telepathy-bugs mailing list