> On 2009-05-09 17:44:20, Aaron Seigo wrote: > > putting things in TimeSource was a way to avoid constant allocations. on > > the one hand, i suppose/hope that people won't be asking for to-the-second > > accurate sun/moon positions from the time engine, but this still seems like > > sort of a step back to where we're doing allocations that probably could be > > re-used every time we want to figure something out. what'd recommend at > > least is adding a Sun* and Moon* pointer to TimeSource and if it's going to > > do sun and moon calcs to allocate them then. tht way they are allocated > > just once. it does mean they hang around even if the request is only > > updated every 10 minutes or whatever. my concern is for rapid updates.
In terms of the ion changes. They're fine to me such that the time engine does not do any connections to the web to get information so that there is no timeout issues being introduced. - Shawn ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/680/#review1099 ----------------------------------------------------------- On 2009-07-06 06:02:10, Petri Damstén wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/680/ > ----------------------------------------------------------- > > (Updated 2009-07-06 06:02:10) > > > Review request for Plasma. > > > Summary > ------- > > Moonrise/set patch using new TimeSource in time dataengine. This also changes > all sun and moon phase calculation code so same code can be used for all the > calculations. This is too late for 4.3, but I'll post it here so it won't get > lost. > > > Diffs > ----- > > /trunk/KDE/kdebase/workspace/plasma/dataengines/weather/ions/CMakeLists.txt > 992178 > > /trunk/KDE/kdebase/workspace/plasma/dataengines/weather/ions/dataengineconsumer.h > PRE-CREATION > /trunk/KDE/kdebase/workspace/plasma/dataengines/weather/ions/ion_bbcukmet.h > 992178 > > /trunk/KDE/kdebase/workspace/plasma/dataengines/weather/ions/ion_bbcukmet.cpp > 992178 > /trunk/KDE/kdebase/workspace/plasma/dataengines/time/solarsystem.h > PRE-CREATION > /trunk/KDE/kdebase/workspace/plasma/dataengines/time/solarsystem.cpp > PRE-CREATION > /trunk/KDE/kdebase/workspace/plasma/dataengines/time/time_solar_export.h > 992178 > /trunk/KDE/kdebase/workspace/plasma/dataengines/time/timesource.h 992178 > /trunk/KDE/kdebase/workspace/plasma/dataengines/time/timesource.cpp 992178 > /trunk/KDE/kdebase/workspace/plasma/dataengines/time/solarposition.h 992178 > /trunk/KDE/kdebase/workspace/plasma/dataengines/time/solarposition.cpp > 992178 > /trunk/KDE/kdebase/workspace/plasma/dataengines/time/CMakeLists.txt 992178 > /trunk/KDE/kdebase/workspace/plasma/dataengines/time/phases.cpp 992178 > > Diff: http://reviewboard.kde.org/r/680/diff > > > Testing > ------- > > > Thanks, > > Petri > > _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
