Review: Approve On Wed, 2012-02-15 at 00:27 +0000, charles wrote: > +#if GTK_CHECK_VERSION(3,0,0) > + GdkRGBA rgba; > + if (gdk_rgba_parse (&rgba, color_spec)) > + gdk_cairo_set_source_rgba (cr, &rgba); > +#else > + GdkColor color; > + if (gdk_color_parse (color_spec, &color)) > + gdk_cairo_set_source_color (cr, > &color); > +#endif
Looks good except it'd be good to fix the whitespace here so that these are the same. status approved review approve -- https://code.launchpad.net/~charlesk/indicator-datetime/fix-gtk3-ftb/+merge/93108 Your team ayatana-commits is subscribed to branch lp:indicator-datetime. _______________________________________________ Mailing list: https://launchpad.net/~ayatana-commits Post to : ayatana-commits@lists.launchpad.net Unsubscribe : https://launchpad.net/~ayatana-commits More help : https://help.launchpad.net/ListHelp