<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/113113/">http://git.reviewboard.kde.org/r/113113/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On October 16th, 2013, 9:23 p.m. UTC, <b>Albert Astals Cid</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;">How hard would be to check on start that not all players are spectators and complain if they are all?</pre>
 </blockquote>




 <p>On October 18th, 2013, 11:51 p.m. UTC, <b>Nemanja Hirsl</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;">Newgamedialog already have some code for enabling/disabling Ok/Cancel buttons. Maybe this code should be improved and not allow game with all spectators? </pre>
 </blockquote>





 <p>On October 19th, 2013, 5:55 p.m. UTC, <b>Albert Astals Cid</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;">Makes sense to me. Alexander?</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;">I implemented this check. If you know how to do it better using signals, please update. :)

During tests I noticed more problems, but I am not sure where they belong to - to the bug tracker or as @todo in the code?

For example, set the neutral player production to any high value (anything above 10) and disable cumulative production. No player will be able to conquer any planet and the game will take forever. If playing without any human player (AI match), the game will "freeze". Or play a game with zero neutral planets - same issue. Or set the starting planets of each player to "vacant".</pre>
<br />










<p>- Alexander</p>


<br />
<p>On October 11th, 2013, 1:31 a.m. UTC, Alexander Schuch wrote:</p>








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

<div>Review request for KDE Games.</div>
<div>By Alexander Schuch.</div>


<p style="color: grey;"><i>Updated Oct. 11, 2013, 1:31 a.m.</i></p>









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


<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;">For AI development it is useful to watch the AIs play. This patch adds a new local human spectator player which does not have any planets and does not count as active player.

Known issues: If all players are spectators, the game will never end.</pre>
  </td>
 </tr>
</table>



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

 <li>CMakeLists.txt <span style="color: grey">(b67b4ba)</span></li>

 <li>dialogs/newgamedlg.cc <span style="color: grey">(7a2924a)</span></li>

 <li>game.cpp <span style="color: grey">(5746b22)</span></li>

 <li>gameview.cc <span style="color: grey">(ebba726)</span></li>

 <li>players/computerplayer.h <span style="color: grey">(29b6b77)</span></li>

 <li>players/computerplayer.cpp <span style="color: grey">(f4fd7a2)</span></li>

 <li>players/localplayer.h <span style="color: grey">(a12b46f)</span></li>

 <li>players/localplayer.cpp <span style="color: grey">(dbb47f3)</span></li>

 <li>players/localplayer_gui.cpp <span style="color: grey">(44a04b5)</span></li>

 <li>players/player.h <span style="color: grey">(a6d91c4)</span></li>

 <li>players/player.cpp <span style="color: grey">(7135127)</span></li>

 <li>players/spectatorplayer.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>players/spectatorplayer.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>players/spectatorplayer_gui.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>players/spectatorplayer_gui.cpp <span style="color: grey">(PRE-CREATION)</span></li>

</ul>

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







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








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