Bug 38559

Summary: Add tp_text_channel_ack_all_pending_messages()
Product: Telepathy Reporter: Guillaume Desmottes <guillaume.desmottes>
Component: tp-glibAssignee: Telepathy bugs list <telepathy-bugs>
Status: RESOLVED FIXED QA Contact: Telepathy bugs list <telepathy-bugs>
Severity: enhancement    
Priority: medium Keywords: patch
Version: unspecified   
Hardware: Other   
OS: All   
URL: http://cgit.collabora.com/git/user/cassidy/telepathy-glib/log/?h=ack-all-38559
Whiteboard: review+
i915 platform: i915 features:
Attachments: add tp_text_channel_ack_all_pending_messages_async() (#38559)

Description Guillaume Desmottes 2011-06-22 03:14:14 UTC
So clients wouldn't have to keep their own list of unacked messages. Furthermore, they should always ack all the messages, even those they don't understand, keeping messages unacked isn't helpful: channel will always be respawned and messages will stay in memory forever.
Comment 2 Guillaume Desmottes 2011-06-22 03:36:35 UTC
Created attachment 48276 [details] [review]
add tp_text_channel_ack_all_pending_messages_async() (#38559)
Comment 3 Jonny Lamb 2011-06-22 03:59:34 UTC
Are we depending on late enough glib for _take_error()? 

+ * Finishes to ack all the pending of messages.

"Finish an asynchronous acknowledgement operation of all messages" I think would be better.

++ otherwise
Comment 4 Guillaume Desmottes 2011-06-22 04:36:33 UTC
(In reply to comment #3)
> Are we depending on late enough glib for _take_error()? 

Yep, that's GLib 2.28 on which we depend already (probably for this function
actually).

> + * Finishes to ack all the pending of messages.
> 
> "Finish an asynchronous acknowledgement operation of all messages" I think
> would be better.

fixed.
Comment 5 Guillaume Desmottes 2011-06-22 05:29:02 UTC
Merged to master; will be in 0.15.3.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.