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 559678 - Cleanups for -DG_DISABLE_DEPRECATED
Cleanups for -DG_DISABLE_DEPRECATED
Status: RESOLVED FIXED
Product: glade
Classification: Applications
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: Glade 3 Maintainers
Glade 3 Maintainers
Depends on:
Blocks: 560423
 
 
Reported: 2008-11-06 23:35 UTC by Kjartan Maraas
Modified: 2009-06-10 22:23 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (1.69 KB, patch)
2008-11-06 23:35 UTC, Kjartan Maraas
none Details | Review

Description Kjartan Maraas 2008-11-06 23:35:23 UTC
Attaching a patch.
Comment 1 Kjartan Maraas 2008-11-06 23:35:43 UTC
Created attachment 122161 [details] [review]
patch
Comment 2 Tristan Van Berkom 2008-11-07 20:34:50 UTC
Look fine and harmless to me, go ahead and commit please.
Comment 3 Kjartan Maraas 2008-11-08 11:26:17 UTC
Commited. Thanks.
Comment 4 Luis Menina 2008-11-12 01:23:24 UTC
It looks like you use the 'n' variable, but never initialize it...
Comment 5 Tristan Van Berkom 2008-11-13 18:39:13 UTC
I refixed this in trunk with:

  len = text ? strlen (text) : 0;

Comment 6 Luis Menina 2008-11-13 21:22:54 UTC
Seems there's also this one:

glade3/gladeui/glade-app.c:361:	prefix = g_win32_get_package_installation_directory (NULL, NULL);
Comment 7 Tristan Van Berkom 2008-11-13 21:41:23 UTC
Ok updated that one too thanks.

As far as plugins go, Im not going to bother much with
the deprecation stuff, those have to build with deprecated 
widgets as a requirement, we fire warnings and stuff about that anyway.
(but the core is ofcourse more important :) )