[Kde-games-devel] Review Request 106772: add option to allow adjacent ships

Roney Gomes roney477 at gmail.com
Sat Mar 9 18:02:12 UTC 2013


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



src/networkentity.cpp
<http://git.reviewboard.kde.org/r/106772/#comment21602>

    Would not be better to work this second feature in a separate patch?


- Roney Gomes


On March 6, 2013, 9:18 p.m., Jaime Torres Amate wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106772/
> -----------------------------------------------------------
> 
> (Updated March 6, 2013, 9:18 p.m.)
> 
> 
> Review request for KDE Games.
> 
> 
> Description
> -------
> 
> Add the option to allow adjacent ships.
> When it is not checked, the ships can not be placed adjacent.
> 
> 
> This addresses bug 168659.
>     http://bugs.kde.org/show_bug.cgi?id=168659
> 
> 
> Diffs
> -----
> 
>   src/battlefield.h ae78166 
>   src/battlefield.cpp 32aa525 
>   src/controller.h bbae7a6 
>   src/controller.cpp 761eda4 
>   src/kbattleship.kcfg b615f30 
>   src/kbattleshipui.rc 7c51ac6 
>   src/mainwindow.cpp 82d9a14 
>   src/message.h 2757cf7 
>   src/message.cpp 59de527 
>   src/networkentity.cpp 5fff5b8 
>   src/playfield.h 56934dc 
>   src/playfield.cpp 61c0ac6 
>   src/protocol.cpp c5cbacc 
>   src/sea.h e76be24 
>   src/sea.cpp de8b697 
>   src/ship.cpp 86858d9 
> 
> Diff: http://git.reviewboard.kde.org/r/106772/diff/
> 
> 
> Testing
> -------
> 
> tested locally.
> 
> 
> Thanks,
> 
> Jaime Torres Amate
> 
>

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


More information about the kde-games-devel mailing list