[Bug 27459] implement a fast-path for Connection properties from spec 0.19.2
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Mon Apr 11 15:26:17 CEST 2011
https://bugs.freedesktop.org/show_bug.cgi?id=27459
--- Comment #4 from Will Thompson <will.thompson at collabora.co.uk> 2011-04-11 06:26:17 PDT ---
http://git.collabora.co.uk/?p=user/oggis/telepathy-glib.git;a=commitdiff;h=f4514c86c30c2cfa5a18de84918af5c8079a96c6
Doesn't TpDBusPropertiesMixin crash (or at least critical) if getting the
GObject property fails?
http://git.collabora.co.uk/?p=user/oggis/telepathy-glib.git;a=commitdiff;h=222bfe4ee068d08c633457906610299f4c2d8eba
I think I'd split the grabbing-things-from-properties block into its own
function, so that you can just return FALSE; when things go awry; this gets rid
of the goto and would make it easier to skim-read.
Test-wise, I guess that the existing tests implicitly exercise the new code
paths, and the tests where you've deliberately broken the properties test the
fallback?
--
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