Review Request 112726: Ordered visibility check for first point of SkyPolyLine in drawSkyPolyline function

Vijay Dhameliya vijay.atwork13 at gmail.com
Fri Oct 18 13:55:33 UTC 2013


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

(Updated Oct. 18, 2013, 1:55 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Edu, KStars, Akarsh Simha, Henry de Valence, and Rafal Kulaga.


Bugs: 303547
    http://bugs.kde.org/show_bug.cgi?id=303547


Repository: kstars


Description
-------

Variable isVisibleLast and isVisible are not initialized so stores random value while operated and with check visibility and correct visibility of point are stored in same variable when passed to toScreen function.  


Diffs
-----

  kstars/skyqpainter.cpp b53b76c 

Diff: http://git.reviewboard.kde.org/r/112726/diff/


Testing
-------

Tested 
Lines on display appears correctly 


Thanks,

Vijay Dhameliya

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20131018/762e7ba0/attachment.html>


More information about the kde-edu mailing list