After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 632789 - [PATCH] tests/icles/: adapted test-colorkey.c and test-xoverlay.c to deprecation of gtk_widget_hide_all
[PATCH] tests/icles/: adapted test-colorkey.c and test-xoverlay.c to deprecat...
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: 0.10.31
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-10-21 12:25 UTC by Florian Gawrilowicz
Modified: 2010-10-21 13:51 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Replaces gtk_widget_hide_all with gtk_widget_hide (1.33 KB, patch)
2010-10-21 12:25 UTC, Florian Gawrilowicz
committed Details | Review

Description Florian Gawrilowicz 2010-10-21 12:25:46 UTC
Created attachment 172931 [details] [review]
Replaces gtk_widget_hide_all with gtk_widget_hide

GTK-2.24 deprecated this function in favor of gtk_widget_hide.
Comment 1 Sebastian Dröge (slomo) 2010-10-21 12:35:43 UTC
Thanks for taking the time to report this bug.
This particular bug has already been reported into our bug tracking system, but we are happy to tell you that the problem has already been fixed. It should be solved in the next software version. You may want to check for a software upgrade.

*** This bug has been marked as a duplicate of bug 625001 ***
Comment 2 Sebastian Dröge (slomo) 2010-10-21 12:37:12 UTC
Oh sorry, I was too fast. Yes, this patch is valid and should be included but use the replacement function only conditionally in an
#if GTK_CHECK_VERSION(x,y,z)
#endif

block if the old gtk_widget_hide() had a different behaviour.
Comment 3 Tim-Philipp Müller 2010-10-21 12:48:01 UTC
I've just changed them all to gtk_widget_hide(). Should be fine. I don't think it matters here anyway, since this is just before shut down. We may just as well remove it.

 commit 8cd8339893743f674aa658459e66af4bccdc8838
 Author: Tim-Philipp Müller <tim.muller@collabora.co.uk>
 Date:   Thu Oct 21 13:38:01 2010 +0100

    tests: don't use deprecated gtk_widget_hide_all()
    
    gtk_widget_hide_all() has been deprecated in gtk+ 2.x and
    removed in 2.9x master. Just use gtk_widget_hide() instead.


Flo: sorry, hadn't seen your patch when I pushed this, bug mail seems to be a bit slow today.