Review Request: Google Code-In: Marble Code Checker Issue Fixes

Geoffrey Mon geofbot at gmail.com
Thu Dec 27 19:01:27 UTC 2012


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

(Updated Dec. 27, 2012, 7:01 p.m.)


Review request for KDE Edu, Marble and Kevin Krammer.


Changes
-------

Diff didn't upload for some reason


Description
-------

For http://www.google-melange.com/gci/task/view/google/gci2012/8127209
31-40 code checker issues (http://www.englishbreakfastnetwork.org/krazy/reports/kde-4.x/kdeedu/marble/index.html) fixed
(mostly pre/postfix increment/decrement issues)


Diffs (updated)
-----

  src/lib/NavigationWidget.cpp dbc2717 
  src/plugins/render/satellites/mex/astrolib.cpp dbd1a86 
  src/plugins/render/satellites/mex/attlib.cpp c9f5486 
  src/plugins/render/satellites/mex/planetarySats.cpp d5dfe66 

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


Testing
-------

None


Thanks,

Geoffrey Mon

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


More information about the kde-edu mailing list