<table><tr><td style="">TallFurryMan added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D24151">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D24151#inline-136712">View Inline</a><span style="color: #4b4d51; font-weight: bold;">mutlaqja</span> wrote in <span style="color: #4b4d51; font-weight: bold;">simclock.cpp:185</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Why was this moved here? Was it tested for any regressions in KStars outside the scheduler?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Because sending the scale update to slots after setting the scale update in SimClock defeats the purpose of sending the scale update. Clients should be able to restore their "real" time values by de-scaling before using the new scale value, like I did in simClockScaleChanged. Else, clients need to store the current scale in order to restore their time values before using the new scale, which is cumbersome.</p>

<p style="padding: 0; margin: 8px;">I verified no code was using the old version of the signal before fixing. Probably this small problem would have been fixed earlier had this signal been actually used in the code.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>BRANCH</strong><div><div>feature__scheduler_synced_to_simulation (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24151">https://phabricator.kde.org/D24151</a></div></div><br /><div><strong>To: </strong>TallFurryMan, wreissenberger, mutlaqja, alexcherney<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>