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

Jaime Torres Amate jtamate at gmail.com
Wed Mar 6 21:18:32 UTC 2013


-----------------------------------------------------------
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.


Changes
-------

Addressed the problems detected.


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 (updated)
-----

  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/20130306/497d2a60/attachment.html>


More information about the kde-games-devel mailing list