<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://git.reviewboard.kde.org/r/106772/">http://git.reviewboard.kde.org/r/106772/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On November 18th, 2012, 6:50 p.m., <b>Roney Gomes</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hi Jaime, your patch works quite nicely, thanks.

Despite your patch perform a good work, I think there's still space for a last improvement. What do you think about notifying the players whether adjacent ships are allowed or not in the chat widget? Something like what we already have about the players' nicknames.

It would be a simple message containing a string like "Adjacent ships are allowed." right after the beginning of a game.</pre>
 </blockquote>







</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hi Roney,

  I'm trying to improve the patch in 4 ways:
* Doing what you say.
* Handle in the right way old navalBattle versions connecting to the new one (right now I've seen two problems: the new one does not know when an old client has connected or the old one (4.9.3) crashes when connects to the new server).
* Check with random data if the protocol handling is right done (security paranoia essential with network communications).
* Do not change the global property for a network game.
</pre>
<br />








<p>- Jaime Torres</p>


<br />
<p>On November 6th, 2012, 12:03 p.m., Jaime Torres Amate wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for KDE Games.</div>
<div>By Jaime Torres Amate.</div>


<p style="color: grey;"><i>Updated Nov. 6, 2012, 12:03 p.m.</i></p>






<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Add the option to allow adjacent ships.
When it is not checked, the ships can not be placed adjacent.</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">tested locally.</pre>
  </td>
 </tr>
</table>



<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>


 <a href="http://bugs.kde.org/show_bug.cgi?id=168659">168659</a>


</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>src/message.cpp <span style="color: grey">(59de527)</span></li>

 <li>src/message.h <span style="color: grey">(2757cf7)</span></li>

 <li>src/mainwindow.cpp <span style="color: grey">(a7cceeb)</span></li>

 <li>src/battlefield.cpp <span style="color: grey">(32aa525)</span></li>

 <li>src/kbattleship.kcfg <span style="color: grey">(b615f30)</span></li>

 <li>src/kbattleshipui.rc <span style="color: grey">(7c51ac6)</span></li>

 <li>src/networkentity.h <span style="color: grey">(c82bfbd)</span></li>

 <li>src/networkentity.cpp <span style="color: grey">(3367389)</span></li>

 <li>src/playfield.h <span style="color: grey">(56934dc)</span></li>

 <li>src/playfield.cpp <span style="color: grey">(61c0ac6)</span></li>

 <li>src/protocol.cpp <span style="color: grey">(791179b)</span></li>

 <li>src/ship.h <span style="color: grey">(5e35f1a)</span></li>

 <li>src/ship.cpp <span style="color: grey">(86858d9)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/106772/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>