Re: [clutter] clutter_label_new_full const params

2007-11-07 Thread Murray Cumming
On Wed, 2007-11-07 at 18:18 +0100, Gwenole Beauchesne wrote: On Wed, 7 Nov 2007, Emmanuele Bassi wrote: Should the third param be '_const_ ClutterColor *color'? It's defined as const in clutter_label_set_color(), so I would expect it to be the same in the constructor. yes, it should

Re: [clutter] clutter_label_new_full const params

2007-11-07 Thread Gwenole Beauchesne
On Wed, 7 Nov 2007, Emmanuele Bassi wrote: Should the third param be '_const_ ClutterColor *color'? It's defined as const in clutter_label_set_color(), so I would expect it to be the same in the constructor. yes, it should be a const, at least for consistency. Oh oh, I was exactly thinking

Re: [clutter] clutter_label_new_full const params

2007-11-07 Thread Emmanuele Bassi
On Wed, 2007-11-07 at 08:05 -0600, Jonathon Jongsma wrote: In the following constructor: ClutterActor* clutter_label_new_full (const gchar *font_name, const gchar *text,

[clutter] clutter_label_new_full const params

2007-11-07 Thread Jonathon Jongsma
In the following constructor: ClutterActor* clutter_label_new_full (const gchar *font_name, const gchar *text, ClutterColor *color); Should the third param be