[Kde-games-devel] Review Request 124392: Remove KExtHighscore classes

Albert Astals Cid aacid at kde.org
Mon Jul 20 17:42:26 UTC 2015


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

(Updated July 20, 2015, 5:42 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Games.


Changes
-------

Submitted with commit 75454239b5ecd9841c1f9427990a9782aa11bf8d by Albert Astals Cid to branch master.


Repository: libkdegames


Description
-------

They are deorecated, complex, unmaintained and offer lots of functionality we don't use and prefer to not maintain.

The classes were also unfinished porting since kreversi is the only user and there's still no KF5 version of it.

Also they are the last source of kdelibs4support in libkdegames so that's a plus.

The only known user of KExtHighscore is kreversi, see https://git.reviewboard.kde.org/r/124393/ for the review request that adds the files there.


Diffs
-----

  CMakeLists.txt 3c211f3 
  KDEGamesConfig.cmake.in 361dbf8 
  highscore/CMakeLists.txt eecb85e 
  highscore/kexthighscore.cpp 3367326 
  highscore/kexthighscore_gui.h 4e8c0fa 
  highscore/kexthighscore_gui.cpp d5dba12 
  highscore/kexthighscore_internal.h 40f301c 
  highscore/kexthighscore_internal.cpp 0bcc62d 
  highscore/kexthighscore_item.h 51c29c5 
  highscore/kexthighscore_item.cpp 9efb069 
  highscore/kexthighscore_tab.h cc96d10 
  highscore/kexthighscore_tab.cpp e6e616b 

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


Testing
-------


Thanks,

Albert Astals Cid

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20150720/f13458a3/attachment.html>


More information about the kde-games-devel mailing list