[Bug 33911] Remove Loudmouth wrapper

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Tue May 3 10:12:33 CEST 2011


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

--- Comment #4 from Jonny Lamb <jonny.lamb at collabora.co.uk> 2011-05-03 01:12:32 PDT ---
(Why can't I copy paste spaces today I wonder?

+  if (vcard_error->domain == WOCKY_XMPP_ERROR)
+    if (vcard_error->code == WOCKY_XMPP_ERROR_BAD_REQUEST ||
+        vcard_error->code == WOCKY_XMPP_ERROR_NOT_ACCEPTABLE)
       tp_error.code = TP_ERROR_INVALID_ARGUMENT;

Can't say I approve of this, but yeah it's not new code.

  DEBUG ("fetching location failed: %s", wocky_error->message);
+ /* FUUCK */
  gabble_set_tp_error_from_wocky (wocky_error, &tp_error);

Hm?

- GError *error = gabble_message_get_xmpp_error (reply);
+ GError *error = NULL;
+
+ /* FIXME: did anything depend on getting errors outside core? */
+ wocky_stanza_extract_errors (reply, NULL, &error, NULL, NULL);

This is not a regression though, is it?

- /* FIXME: add a helper in error.c to extract the type, error, and message
- * from an XMPP stanza.
- */

Woo!

Basically fine though.

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



More information about the telepathy-bugs mailing list