It seems you leak this list and it's elements here. Each element of the GList should be g_free'd, and the list should be g_list_free'd once you've finished with it.
In fact, it seems that there is already a convenience function in mn-util.c for doing this (mn_g_list_free_deep_custom), so you should use this to free your list.
+ public bool can_use_actions(void)
+ {
+ /* Returns True if notification server will make use of actions, False if not */
+ return mn_popups_server_caps_actions;
+ }
This function should really return a gboolean, to keep it in the same style as the existing code.
Thank you for your work there. Some comments on the patch though:
- self_add_ actions( self); can_use_ actions == 1 ) actions( self);
+ if ( mn_popups_
+ self_add_
This should be "if ( mn_popups_ can_use_ actions (self) == 1 )" shouldn't it?
+ mn_popups_ server_ caps = notify_ get_server_ caps();
It seems you leak this list and it's elements here. Each element of the GList should be g_free'd, and the list should be g_list_free'd once you've finished with it.
In fact, it seems that there is already a convenience function in mn-util.c for doing this (mn_g_list_ free_deep_ custom) , so you should use this to free your list.
+ public bool can_use_ actions( void) server_ caps_actions;
+ {
+ /* Returns True if notification server will make use of actions, False if not */
+ return mn_popups_
+ }
This function should really return a gboolean, to keep it in the same style as the existing code.
+ /* Notify-OSD */ get_server_ info(¬ify_ name, ¬ify_vendor, ¬ify_version, ¬ify_ spec_version) ) set_sensitive( selfp-> popups_ position_ section_ label, FALSE); set_sensitive( selfp-> popups_ position_ attached_ radio, FALSE); set_sensitive( selfp-> popups_ position_ free_radio, FALSE); set_sensitive( selfp-> popups_ expiration_ section_ label, FALSE); set_sensitive( selfp-> popups_ expiration_ default_ radio, FALSE); set_sensitive( selfp-> popups_ expiration_ never_radio, FALSE); set_sensitive( selfp-> popups_ expiration_ after_radio, FALSE); set_sensitive( selfp-> popups_ expiration_ scale, FALSE);
+ if (notify_
+ {
+ if (! strcmp(notify_name, "notify-osd"))
+ {
+ gtk_widget_
+ gtk_widget_
+ gtk_widget_
+ gtk_widget_
+ gtk_widget_
+ gtk_widget_
+ gtk_widget_
+ gtk_widget_
+ }
+ }
It seems you just make these insensitive, but the spec says that they should not be visible.
Thanks again for your effort there :)