XfceAboutDialog SVN Patch

[Jeff Franks]

Jeff Franks wrote:

> Benedikt,
>
> I mentioned previously about the lack of a getter and setter for the 
> XfceAboutDialog 'icon' property. There are a few reasons to provide 
> these. Firstly, there are getters and setters for all the other 
> properties, why not the icon. Secondly, there is a 
> xfce_about_dialog_new_empty() function so it would be consistent to 
> provide an icon getter and setter that can be called in when using 
> this function.
>
> Lastly, and most important for me, it makes it easier for language 
> bindings. I know I could use g_object_set() but that is an off 
> function in C++ and one best avoided, unless there is no choice. I 
> only need the setter but there should be a getter for consistency. I 
> have attached a patch that adds these two functions:
>
> void xfce_about_dialog_set_icon(XfceAboutDialog* dialog, GdkPixbuf 
> *icon);
> GdkPixbuf   *xfce_about_dialog_get_icon(XfceAboutDialog* dialog);
> The patch also updates  xfce_about_dialog_get_property() and 
> xfce_about_dialog_new_with_values() to call the new 
> xfce_about_dialog_set_icon().
>
I meant xfce_about_dialog_set_property().

Jeff