Bug 28682 - Fetch Account property each time we receive a message
Summary: Fetch Account property each time we receive a message
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: logger (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Guillaume Desmottes
QA Contact: Telepathy bugs list
URL: http://git.collabora.co.uk/?p=user/ca...
Whiteboard:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2010-06-23 03:15 UTC by Guillaume Desmottes
Modified: 2010-06-23 03:50 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Description Guillaume Desmottes 2010-06-23 03:15:00 UTC
Each time a message is logged, a new TpAccount is created and to the Account property are fetched by tp-glib. We should reuse the existing TpAccount instead.
Comment 2 Simon McVittie 2010-06-23 03:37:20 UTC
> +#include <telepathy-glib/account.h>
>  #include <telepathy-glib/util.h>

Might as well be <telepathy-glib/telepathy-glib.h>? (This applies to entry.[ch].)

> + * Returns: the path as the #TplEntry:account property

"...: the object path of the #..."

Also, if you add "<!-- no more to say -->" as the "body" of the doc-comment for every method whose "Returns:" is enough documentation, gtk-doc will consider it to be documented, and shut up. This will be useful when we try to get Tpl fully-documented.

> + * tpl_entry_get_account

Similarly, add "<!-- no more to say -->".
Comment 3 Guillaume Desmottes 2010-06-23 03:43:41 UTC
(In reply to comment #2)
> > +#include <telepathy-glib/account.h>
> >  #include <telepathy-glib/util.h>
> 
> Might as well be <telepathy-glib/telepathy-glib.h>? (This applies to
> entry.[ch].)

done.

> > + * Returns: the path as the #TplEntry:account property
> 
> "...: the object path of the #..."
> 
> Also, if you add "<!-- no more to say -->" as the "body" of the doc-comment for
> every method whose "Returns:" is enough documentation, gtk-doc will consider it
> to be documented, and shut up. This will be useful when we try to get Tpl
> fully-documented.
> 
> > + * tpl_entry_get_account
> 
> Similarly, add "<!-- no more to say -->".

done

(I amend the commits).
Comment 4 Guillaume Desmottes 2010-06-23 03:50:06 UTC
Merged.


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.