On Wed, 2007-03-28 at 19:56 -0400, Steven Walter wrote: > Comments on this code? I'm not terribly familiar with the rhythmbox > code, so please let me know if there's anything in it that would keep it > from being applied.
Looking at the patch on http://bugzilla.gnome.org/show_bug.cgi?id=424086, Christophe's comment about storing a reference to the playlist source, rather than finding it each time, is the big issue. Apart from that it looks pretty good, and I'd be happy to commit it to svn. Cheers, James "Doc" Livingston -- "This struct already has a perfectly good definition" -- MPW C error message _______________________________________________ rhythmbox-devel mailing list [email protected] http://mail.gnome.org/mailman/listinfo/rhythmbox-devel
