[Bug 34932] Use WockyMetaPorter
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Wed Mar 23 17:16:25 CET 2011
https://bugs.freedesktop.org/show_bug.cgi?id=34932
--- Comment #4 from Will Thompson <will.thompson at collabora.co.uk> 2011-03-23 09:16:24 PDT ---
Your fixes to my first batch of comments look good, except:
(In reply to comment #2)
> (In reply to comment #1)
> > + addresses = g_list_append (addresses, socket_address);
> >
> > <http://en.wikipedia.org/wiki/Schlemiel_the_Painter's_algorithm>. Yeah, it's
> > not a big deal here, but this is one reason I like stack-allocated GQueues: you
> > can build GLists with repeated O(1) append. I guess this isn't new code though.
>
> Yeah yeah, I've never been a fan of this and actually I'm not sure why. I guess
> perhaps because it feels like such an epic hack to get around the terrible data
> structure we're using. I guess we're doing worse things just by using GObject.
>
> Fixed.
--. Repeated prepends followed by reverse is even worse than using a dumb O(n²)
algorithm.
--
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