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 361362 - gnome-dateedit.h should include time.h
gnome-dateedit.h should include time.h
Status: VERIFIED FIXED
Product: libgnomeui
Classification: Deprecated
Component: general
CVS HEAD
Other All
: Normal normal
: future
Assigned To: libgnomeui maintainers
libgnomeui maintainers
Depends on:
Blocks:
 
 
Reported: 2006-10-11 06:59 UTC by Jose Dapena Paz
Modified: 2006-10-24 08:28 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16



Description Jose Dapena Paz 2006-10-11 06:59:59 UTC
Please describe the problem:
The gnome-dateedit.h header uses time_t, but it does not include it. It forces the developer to include time.h when he includes gnome-dateedit.h, and it shouldn't happen, as all headers should include the symbols they use.

Steps to reproduce:
Just compile a piece of code with:
#include <gnome-date-edit.h>

and not including time.h.

Actual results:
It returns a warning (preventing successful compilation with -Werror).

Expected results:
No warnings

Does this happen every time?
In gcc3.

Other information:
The fix is simple: just add #include <time.h> to the gnome-dateedit.h.
Comment 1 Kjartan Maraas 2006-10-23 14:14:28 UTC
Fixed. Thanks.