misplaced #include <time.h> in the KDE frontend
Posted: 08.05.2003, 18:06
by salo
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
Posted: 12.05.2003, 22:14
by Christophe
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.
Posted: 13.05.2003, 18:33
by salo
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.