[PATCH 3/3] Compile subsurface with gtk3

[Dirk Hohndel]

On Jan 28, 2013, at 7:55 AM, Linus Torvalds wrote:

> Oh christ. Some over-eager search-and-replace caused this moronic end result:
> 
> On Sun, Jan 27, 2013 at 4:54 PM, Linus Torvalds
> <torvalds at linux-foundation.org> wrote:
>> 
>> +/* Gtk made this "simpler" */
>> +const char *get_active_text(GtkComboBox *combo_box)
>> +{
>> +       return get_active_text(combo_box);
>> +}
> 
> Which is obviously bogus.
> 
> Explanation: it *should* have done
> 
>    return gtk_entry_get_text(GTK_ENTRY(gtk_bin_get_child(GTK_BIN(combo_box))));
> 
> but because that's such an ugly pattern, I created that
> "get_active_text()" helper function, and then did a search-and-replace
> of that horribly ugly pattern with just "get_active_text()".
> 
> But it replaced the text inside the helper function too, so how the
> helper function just called itself rather than do the right thing. And
> this is mainly used for some equipment stuff, so I didn't notice.
> 
> I'll re-do the series, but I'll test some more first.

BTW: I pushed the first two out already. They seemed harmless enough.

/D
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4130 bytes
Desc: not available
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20130128/e693c818/attachment-0001.bin>