[Kstars-devel] Review Request 110973: Update the local time before we update the timebox.
Aneesh Dogra
aneesh at anee.me
Thu Jun 13 13:24:48 UTC 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110973/
-----------------------------------------------------------
(Updated June 13, 2013, 6:54 p.m.)
Review request for KStars.
Changes
-------
Fix the bug links.
Description
-------
The time in the timebox lags by one step when the user advances one step backward/forward in time. The happens because the timebox updater function and local time updater function are connected with the same signal i.e timeChanged. The order of connection to this signal is reversed. We first connect the timebox updater and then the local time updater. This causes the timebox to be updated before the local time; hence, causing the lag. The patch simply uses the correct order, i.e connects the local time updater to the signal before it connects the timebox. This ensures that local time gets updated before the timebox.
This addresses bugs 296829 and 321028.
http://bugs.kde.org/show_bug.cgi?id=296829
http://bugs.kde.org/show_bug.cgi?id=321028
Diffs
-----
kstars/kstarsinit.cpp f89bedd
Diff: http://git.reviewboard.kde.org/r/110973/diff/
Testing
-------
Thanks,
Aneesh Dogra
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kstars-devel/attachments/20130613/3cf20645/attachment.html>
More information about the Kstars-devel
mailing list