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

Reply via email to