Review Request 121789: More cleanup of cpp code this time.
Jeremy Whiting
jpwhiting at kde.org
Fri Jan 2 16:25:15 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121789/
-----------------------------------------------------------
(Updated Jan. 2, 2015, 4:25 p.m.)
Status
------
This change has been marked as submitted.
Review request for KDE Edu and Rahul Chowdhury.
Repository: khangman
Description
-------
Remove duplicate variables for word and current answer.
Rename setLanguage to scanLanguages to explain what it does.
Clean up includes in khangman.cpp.
Initialize m_currentLanguage and m_currentCategory.
Remove level concept, since we use category everywhere instead.
Remove unused private variables.
Add handling of ' - space and middot characters as the old desktop code had.
Remove a couple settings we no longer need or use.
Diffs
-----
src/khangman.h 13f2b1696e39a7c0b3db8e86755a2e9a1630220d
src/khangman.cpp 1e8e923a1175643ea190405d088d14ca26ffa004
src/khangman.kcfg 44bf6fe209140cb358b97c4b9c96a2c8651be8fa
src/qml/GamePage.qml 517673226d80dc3f491d3f7e51436f2f534f8989
Diff: https://git.reviewboard.kde.org/r/121789/diff/
Testing
-------
It builds, runs and no longer crashes on start when there's no ~/.config/khangmanrc file.
Thanks,
Jeremy Whiting
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20150102/df0b4c1d/attachment.html>
More information about the kde-edu
mailing list