On Fri, 05 Apr 2013 17:43:42 -0700 Matthew Brush mbrush@codebrainz.ca wrote:
On 14-04-05 11:40 AM, Dimitar Zhekov wrote:
On Fri, 5 Apr 2013 08:58:21 +0200 Quentin Glidic sardemff7+geany@sardemff7.net wrote:
geanyinsertnum/src/insertnum.c | 4 ++--
- GTK_WIDGET_SET_FLAGS(button, GTK_CAN_DEFAULT);
- gtk_widget_set_can_default(button, TRUE);
geanyinsertnum/src/insertnum.c | 14 +++++++-------
- combo = GTK_COMBO_BOX(gtk_combo_box_entry_new_text());
- combo = GTK_COMBO_BOX(gtk_combo_box_text_new_with_entry());
- gtk_combo_box_append_text(combo, "2");
- gtk_combo_box_text_append_text(GTK_COMBO_BOX_TEXT(combo), "2");
...
Thank you, Quentin, but...
Using gtkcompat requires either Geany >= 20130818 or gtk+ >= 2.24.
To the lead developer who reviews the gtkcompat patches: please do not apply the ones for insertnum.
Rather than breaking compat with older Geany or preventing GTK3 users from using the plugin, you could just add the 2 #if's needed to make the plugin build with GTK3 for now.
The above are only examples - there are actually 4 #if's, and more to avoid the deprecated symbols. And it would be easier without gtkcompat, which only muddles the water in this case. But I'll probably be able to get rid of it by #defining GTKCOMPAT_H before including geanyplugin.h.