[Kstars-devel] Review Request 110973: Update the local time before we update the timebox.

Aneesh Dogra aneesh at anee.me
Thu Jun 13 13:22:53 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110973/
-----------------------------------------------------------

(Updated June 13, 2013, 6:52 p.m.)


Review request for KStars.


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 https://bugs.kde.org/show_bug.cgi?id=296829 and https://bugs.kde.org/show_bug.cgi?id=321028.
    http://bugs.kde.org/show_bug.cgi?id=https://bugs.kde.org/show_bug.cgi?id=296829
    http://bugs.kde.org/show_bug.cgi?id=https://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/1c838e93/attachment.html>


More information about the Kstars-devel mailing list