misplaced #include <time.h> in the KDE frontend

Report bugs, bug fixes and workarounds here.
Topic author
salo

misplaced #include <time.h> in the KDE frontend

Post #1by salo » 08.05.2003, 18:06

src/celestia/kde/kdeglwidget.cpp requires localtime() definition from time.h but that is only included from src/celestia/kde/kdepreferencesdialog.cpp. move the inclusion to src/celestia/kde/kdeapp.h which is included by both of them.

patch for celestia-1.3.0 sources: http://Xtrmntr.org/salo/celestia-1.3.0-time.h.diff

Christophe
Developer
Posts: 944
Joined: 18.07.2002
With us: 22 years 4 months
Location: Lyon (France)

Post #2by Christophe » 12.05.2003, 22:14

Once again thanks for your patch Salo, it's been commited.

In the future it'd probably best for you to post your patch on the developers list (celestia-developers@sourceforge.net), see here to join. I sometimes miss posts in the forum, but I never miss any post on the list.
Christophe

Topic author
salo

Post #3by salo » 13.05.2003, 18:33

ok, will do. thanks for the heads-up. apparently i was not thorough enough, i see you have sf.net project (i was accessing the web page at shatters.net directly) even with accessible cvs repository. i'll make sure the (possible) patches i submit later will apply to the cvs trunk.


Return to “Bugs”