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 572236 - [fixed by ifdef] Remove deprecated Glib symbols
[fixed by ifdef] Remove deprecated Glib symbols
Status: VERIFIED INVALID
Product: Gnumeric
Classification: Applications
Component: Compilation
git master
Other All
: Normal normal
: ---
Assigned To: Jody Goldberg
Jody Goldberg
Depends on:
Blocks: 560423
 
 
Reported: 2009-02-18 04:08 UTC by André Klapper
Modified: 2009-06-10 22:24 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description André Klapper 2009-02-18 04:08:53 UTC 
http://live.gnome.org/GnomeGoals/RemoveDeprecatedSymbols/Glib

./src/tools/dao.c:	g_date_set_time_t (&date, now);
./src/tools/dao.c: 	g_date_set_time (&date, t.tv_sec);
./configure.in:AC_CHECK_FUNCS(g_date_set_time_t g_slice_alloc pango_font_map_create_context gtk_recent_manager_get_default gsf_open_pkg_foreach_rel gsf_infile_msvba_steal_modules)


http://library.gnome.org/devel/glib/unstable/glib-Date-and-Time-Functions.html#g-date-set-time
Comment 1 Morten Welinder 2009-02-18 09:38:04 UTC 
There is nothing wrong with that code.  In particular, it does not use
deprecated symbols.
Comment 2 André Klapper 2009-02-18 11:17:56 UTC 
Sigh... OK, ifdef'ed. Thanks.

990 	#ifdef HAVE_G_DATE_SET_TIME_T
991 	now = time (NULL);
992 	g_date_set_time_t (&date, now);
993 	#else
994 	GTimeVal t;
995 	g_get_current_time (&t);
996 	now = t.tv_sec;
997 	g_date_set_time (&date, t.tv_sec);
Comment 3 Christian Kirbach 2009-04-15 22:00:14 UTC 
I will go ahead and set gnumeric to "not needed" for this goal.