[Bug 54879] [next] initial 'next' branch
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Thu Sep 13 18:45:02 CEST 2012
https://bugs.freedesktop.org/show_bug.cgi?id=54879
--- Comment #7 from Simon McVittie <simon.mcvittie at collabora.co.uk> 2012-09-13 16:45:02 UTC ---
In "service-point: stop using deprecated APIs and add a small test"
+ g_print ("adding %s to the string list\n", *ptr);
I think you left your debugging code in.
I think the complexity of this patch is an indication that Conn.I.ServicePoint
should change like this on next:
Service_Point: (us) type, ID -> (uus) type, handle, ID
Service_Point_Info: ((us)as) type, ID, aliases -> ((uus)a{su}) or something
I've only skim-read the patch, but it looks basically sensible, and the fact
that you add a test is a good sign.
+# MC used to critical if more than one emergency service point was
+# given by the CM. That's silly, so let's test it.
Almost as if this had never been tested...
--
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
More information about the telepathy-bugs
mailing list