[Bug 20942] tp_asv_new() and tp_asv_set_*

[bugzilla-daemon at freedesktop.org]

http://bugs.freedesktop.org/show_bug.cgi?id=20942





--- Comment #2 from Will Thompson <will.thompson at collabora.co.uk>  2009-03-30 06:50:32 PST ---
Coding style: Telepathy style for wrapping function calls is:

  foo_get_bar (very, long, argument, list, ...,
      four_space_indent);

I note that
http://git.collabora.co.uk/?p=user/davyd/telepathy-glib-davyd.git;a=commitdiff;h=904a2891a96d4ac56128cf9766ac7d9386e53481
"fixes" the indentation still to be wrong. :)

>From the perspective of helping people not get this wrong, I'd replace:

  "a #GHashTable for storing a{sv} maps"

with:

  "a #GHashTable created with tp_asv_new()"

If people want to make their own hash table, they can read the tp_asv_new()
documentation and discover what they must do.


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