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

Jaime Torres Amate jtamate at gmail.com
Tue Nov 6 12:03:02 UTC 2012


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

(Updated Nov. 6, 2012, 12:03 p.m.)


Review request for KDE Games.


Changes
-------

Now the game type can not be modified in the middle of a game (easy).
Now the rule is enforced in a network game (not so easy) with a new message type.


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/message.cpp 59de527 
  src/message.h 2757cf7 
  src/mainwindow.cpp a7cceeb 
  src/battlefield.cpp 32aa525 
  src/kbattleship.kcfg b615f30 
  src/kbattleshipui.rc 7c51ac6 
  src/networkentity.h c82bfbd 
  src/networkentity.cpp 3367389 
  src/playfield.h 56934dc 
  src/playfield.cpp 61c0ac6 
  src/protocol.cpp 791179b 
  src/ship.h 5e35f1a 
  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/20121106/37c938a7/attachment.html>


More information about the kde-games-devel mailing list