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 694362 - [PATCH] Fix some deprecated declarations
[PATCH] Fix some deprecated declarations
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-02-21 14:24 UTC by Ángel Guzmán Maeso (shakaran)
Modified: 2013-02-26 16:16 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix some deprecated declarations (4.94 KB, patch)
2013-02-21 14:24 UTC, Ángel Guzmán Maeso (shakaran)
none Details | Review
Panel patches splitted as requested by irc (10.00 KB, application/octet-stream)
2013-02-21 14:49 UTC, Ángel Guzmán Maeso (shakaran)
  Details

Description Ángel Guzmán Maeso (shakaran) 2013-02-21 14:24:17 UTC
Created attachment 237042 [details] [review]
Fix some deprecated declarations

* gui_gtk.c: Fix deprecated declarations gtk_icon_info_free to
g_object_unref and gtk_window_set_opacity -> gtk_widget_set_opacity
using as GTK_WIDGET instead of GTK_WINDOW
See https://mail.gnome.org/archives/commits-list/2013-February/msg02049.html

* um-utils.c: Fix deprecated declarations gtk_icon_info_free to
g_object_unref

* scrollarea.c: Fix deprecated declarations
gdk_window_get_pointer to gdk_window_get_device_position

* cc-power-panel.c: Fix deprecated declarations
gtk_scrolled_window_add_with_viewport to gtk_container_add
http://www.mail-archive.com/crash-catcher@lists.fedorahosted.org/msg03480.html

* cc-input-chooser.c: Fix maybe uninitialized warning with widget and
initial var
Comment 1 Ángel Guzmán Maeso (shakaran) 2013-02-21 14:49:50 UTC
Created attachment 237044 [details]
Panel patches splitted as requested by irc

Panel patches splitted as requested by irc
Comment 2 Bastien Nocera 2013-02-21 16:08:44 UTC
Comment on attachment 237044 [details]
Panel patches splitted as requested by irc

A tar.gz file isn't a "patch" in the bugzilla sense.
Comment 3 Bastien Nocera 2013-02-21 16:17:16 UTC
I didn't commit the patch about scrollarea.c as I'm not happy about the fact that the event is completely disconnected from the mouse pointer event itself.
The rest is committed with the commit messages fixed.
Comment 4 Ángel Guzmán Maeso (shakaran) 2013-02-22 08:09:20 UTC
what should be the proper fix for scrollarea.c then? It is the only way to remove gdk_window_get_pointer() that I know. If it is not fixed, we still get the deprecation warning at compilation time.

For example, here are using the same trick: http://web.archiveorange.com/archive/v/bZBuSlfX3HWzorhZyMTo
Comment 5 Bastien Nocera 2013-02-26 16:16:14 UTC
(In reply to comment #4)
> what should be the proper fix for scrollarea.c then? It is the only way to
> remove gdk_window_get_pointer() that I know.

The pointer position is supposed to come from a mouse event.

> If it is not fixed, we still get
> the deprecation warning at compilation time.

Yes, but we have until GTK+ 4.0 to fix it.