[Kde-games-devel] Review Request 126124: Port to KF5/QT5

[Christian Ehrlicher]

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



CMakeLists.txt (line 56)
<https://git.reviewboard.kde.org/r/126124/#comment61577>

    I think the double quotes are not needed here (and maybe counterproductive because cmake treats this as one argument)



src/board.cpp (line 711)
<https://git.reviewboard.kde.org/r/126124/#comment61576>

    The dialog should be created on the heap, not the stack, see https://blogs.kde.org/node/3919



src/main.cpp (line 72)
<https://git.reviewboard.kde.org/r/126124/#comment61578>

    krazy: use isEmpty()


- Christian Ehrlicher


On Dez. 21, 2015, 5:30 nachm., Manoharan Sundaramoorthy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126124/
> -----------------------------------------------------------
> 
> (Updated Dez. 21, 2015, 5:30 nachm.)
> 
> 
> Review request for KDE Games, Knights and Miha Čančula.
> 
> 
> Repository: knights
> 
> 
> Description
> -------
> 
> Knights: Port to KF5/QT5
> 
> 
> Diffs
> -----
> 
>   icons/48-apps-knights.png PRE-CREATION 
>   icons/16-apps-knights.png PRE-CREATION 
>   icons/32-apps-knights.png PRE-CREATION 
>   doc/CMakeLists.txt 5aad7c2 
>   CMakeLists.txt ed25a68 
>   icons/64-apps-knights.png PRE-CREATION 
>   icons/CMakeLists.txt 7affd27 
>   icons/hi16-app-knights.png ee4a3d9 
>   icons/hi32-app-knights.png 120c7b2 
>   icons/hi48-app-knights.png 26684f0 
>   icons/hi64-app-knights.png df75196 
>   src/CMakeLists.txt 6d77801 
>   src/board.h d464bd1 
>   src/board.cpp 775cfd8 
>   src/clock.h 340c885 
>   src/clock.cpp 1e13366 
>   src/clockwidget.h 5686a95 
>   src/clockwidget.cpp 2a4e744 
>   src/core/item.cpp 30fe74e 
>   src/core/move.h 50e32b0 
>   src/core/move.cpp 4f7ca0e 
>   src/core/piece.cpp 4564ae8 
>   src/core/pos.h 343a8b5 
>   src/core/pos.cpp 8623c27 
>   src/customdifficultydialog.ui f9f7b29 
>   src/enginesettings.h 2942ad1 
>   src/enginesettings.cpp 4802ce0 
>   src/enginesettings.ui eb59af2 
>   src/externalcontrol.h 40dbf55 
>   src/externalcontrol.cpp 1467491 
>   src/gamedialog.h 5518290 
>   src/gamedialog.cpp 2d0a124 
>   src/gamedialog.ui 436887d 
>   src/gamemanager.h fe7f883 
>   src/gamemanager.cpp c9d9566 
>   src/historywidget.h 0d9f823 
>   src/historywidget.cpp 9f7f3fd 
>   src/historywidget.ui aacc0be 
>   src/knights.cpp cb688a7 
>   src/knights.desktop 400cf57 
>   src/knightsdebug.h PRE-CREATION 
>   src/knightsview.h 40df529 
>   src/knightsview.cpp 5451eb9 
>   src/main.cpp 504fcc5 
>   src/offerwidget.cpp 0993ae2 
>   src/org.kde.knights.desktop PRE-CREATION 
>   src/popup.ui d4665af 
>   src/promotiondialog.ui 810e67d 
>   src/proto/chatwidget.h 4afcdf6 
>   src/proto/chatwidget.cpp fa209f9 
>   src/proto/chatwidget.ui 82ff8b7 
>   src/proto/computerprotocol.cpp a0b032f 
>   src/proto/ficsdialog.h 93d5564 
>   src/proto/ficsdialog.cpp d89a07c 
>   src/proto/ficsdialog.ui 8eea2e7 
>   src/proto/ficsprotocol.h 3d3e395 
>   src/proto/ficsprotocol.cpp 2c9b07d 
>   src/proto/localprotocol.cpp 711ec5c 
>   src/proto/protocol.cpp e489d33 
>   src/proto/seekgraph.h cba3b5f 
>   src/proto/seekgraph.cpp 3c83591 
>   src/proto/textprotocol.h 8216f24 
>   src/proto/textprotocol.cpp d6f5290 
>   src/proto/uciprotocol.cpp 69d7efb 
>   src/proto/xboardprotocol.cpp 01fb4ae 
>   src/rules/chessrules.cpp 3e993e4 
>   src/rules/rules.h 032cde4 
> 
> Diff: https://git.reviewboard.kde.org/r/126124/diff/
> 
> 
> Testing
> -------
> 
> 1. Tested Playing against gnuchess
> 2. Two human players
> 3. Playing with clock
> 
> 
> Thanks,
> 
> Manoharan Sundaramoorthy
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20151221/829cc867/attachment-0001.html>