[Marble-devel] Review Request 114782: Refactoring planet rendering code in Stars Plugin

[Illya Kovalevskyy]

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

(Updated Jan. 1, 2014, 8:18 p.m.)


Status
------

This change has been marked as submitted.


Review request for Marble, Dennis Nienhüser, Torsten Rahn, and René Küttner.


Repository: marble


Description
-------

According to Google Code-in 2013 task (http://www.google-melange.com/gci/task/view/google/gci2013/5777453708476416):

* Planet rendering moved to renderPlanet(..)
* Planet rendering flags merged into QMap<QString, bool> m_renderPlanet (<planet name, available?>)
* Settings loading/saving, reading/writing adjusted to serialize/unser. m_renderPlanet


Diffs
-----

  src/plugins/render/stars/StarsPlugin.h a027ef3 
  src/plugins/render/stars/StarsPlugin.cpp 89fd1a8 

Diff: https://git.reviewboard.kde.org/r/114782/diff/


Testing
-------

Compared to KStars, GUI testing


Thanks,

Illya Kovalevskyy

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20140101/60ea9a6f/attachment-0001.html>