[Marble-devel] Review Request 114719: [GCI] Factor constellation names out of star plugin. (1)

[Commit Hook]

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


This review has been submitted with commit 0dbfcc0132b19164528afe7159879dea55e854a8 by Mihail Ivchenko to branch master.

- Commit Hook


On Dec. 29, 2013, 10:49 a.m., Mihail Ivchenko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/114719/
> -----------------------------------------------------------
> 
> (Updated Dec. 29, 2013, 10:49 a.m.)
> 
> 
> Review request for Marble.
> 
> 
> Repository: marble
> 
> 
> Description
> -------
> 
> Factor constellation names out of star plugin.
> That's first way to do that.
> GCI task: http://www.google-melange.com/gci/task/view/google/gci2013/5233734269796352
> 
> Second way: https://git.reviewboard.kde.org/r/114720
> 
> That's way takes 15-16 ms for parsing csv (first run) and 5 ms with cached file.
> 
> I prefer this way, because csv is easier to edit and speed is not very different.
> 
> 
> Diffs
> -----
> 
>   src/plugins/render/stars/StarsPlugin.cpp 4bed657 
>   data/stars/names.csv PRE-CREATION 
>   data/CMakeLists.txt 9ac9620 
>   Messages.sh 93fac6b 
> 
> Diff: https://git.reviewboard.kde.org/r/114719/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Mihail Ivchenko
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20131230/ef28b441/attachment.html>