[Kde-games-devel] Review Request: Port all KNewStuff2 code in KDEGames to KNewStuff3
Parker Coates
parker.coates at kdemail.net
Sun Mar 21 22:05:42 CET 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/3343/
-----------------------------------------------------------
Review request for KDE Games.
Summary
-------
I just went through KDEGames trunk and updated all uses of KNewStuff2 to KNewStuff3. Most applications use KGameThemeSelector, so only their knsrc files needed updating.
Summary of changes:
KAtomic: Updated custom level set dialog. Had to work around a bug in the KNS3 interface.
KBlocks: Uses KGameThemeSelector. Updated knsrc file.
KDiamond: Uses KGameThemeSelector. Updated knsrc file.
KGoldrunner: Has a knsrc file, but doesn't seem to use it anywhere. Updated it anyway.
Kigo: Has two knsrc files: one for saved games, one for themes. Unfortunately, both of them use the same provider URL, so the Get More Themes button in the theme selector lets you download saved games to the themes directory, which is obviously quite confusing. So I disable the button in the theme selector until that is sorted out.
KMines: Has an invalid, unused knsrc file. Updated it anyway.
KTron: Uses KGameThemeSelector. Updated knsrc file. (There doesn't seem to be anything on the server for it, though.)
KGameThemeSelector: Updated to KNewStuff3.
Diffs
-----
trunk/KDE/kdegames/katomic/chooselevelsetdialog.h 1106088
trunk/KDE/kdegames/katomic/chooselevelsetdialog.cpp 1106088
trunk/KDE/kdegames/katomic/katomic.knsrc 1106088
trunk/KDE/kdegames/katomic/levelsetwidget.ui 1106088
trunk/KDE/kdegames/kblocks/kblocks.knsrc 1106088
trunk/KDE/kdegames/kdiamond/src/kdiamond.knsrc 1106088
trunk/KDE/kdegames/kgoldrunner/themes/kgoldrunner.knsrc 1106088
trunk/KDE/kdegames/kigo/src/gui/mainwindow.cpp 1106088
trunk/KDE/kdegames/kmines/kmines.knsrc 1106088
trunk/KDE/kdegames/ktron/ktron.knsrc 1106088
trunk/KDE/kdegames/libkdegames/CMakeLists.txt 1106088
trunk/KDE/kdegames/libkdegames/kgamethemeselector.cpp 1106088
Diff: http://reviewboard.kde.org/r/3343/diff
Testing
-------
Installed and uninstalled a theme for all those games that support it.
Thanks,
Parker
More information about the kde-games-devel
mailing list