<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/112676/">http://git.reviewboard.kde.org/r/112676/</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 8th, 2013, 8:21 p.m. UTC, <b>Nemanja Hirsl</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  



<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
 <thead>
  <tr>
   <th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
    <a href="http://git.reviewboard.kde.org/r/112676/diff/3/?file=199906#file199906line36" style="color: black; font-weight: bold; text-decoration: underline;">players/ai/example/example.cpp</a>
    <span style="font-weight: normal;">

     (Diff revision 3)

    </span>
   </th>
  </tr>
 </thead>



 
 

 <tbody>

  <tr>
    <th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
    <th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">36</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="n">emit</span><span class="p">(</span><span class="n">donePlaying</span><span class="p">());</span></pre></td>
  </tr>

 </tbody>

</table>

  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">AiExample will not play the game - it will just pass the turn.
This is not stated anywhere and might be confusing for human players.

Do we really need it in release configuration? Maybe to keep it in debug config if necessary?</pre>
 </blockquote>



 <p>On October 9th, 2013, 1:16 a.m. UTC, <b>Alexander Schuch</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;">I personally like the idea to add a textual description for humans next to the selected player controller. Information like that can be stated there. But right now I really lack knowledge about how to add new GUI elements. In the source, I already added that description as class description in example.h:25.

So for now I propose to simply comment out adding that AI player controller in NewGameDlg::NewGameDlg(). The code can still be compiled, so CMakeLists.txt does not need to be changed anymore. The compiler should notice that this class is not referenced and as such should not add the code to the binary. Right?

Or what do you mean by "debug config"? I ran "ccmake" and the only configuration option roughly named that way is called "CMAKE_BUILD_TYPE" which can be set to "Debug".</pre>
 </blockquote>







</blockquote>
<pre style="margin-left: 1em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Adding description gui element next to player might be a good task for future. For now we can remove it (comment out creation of this player) from NewGameDlg::NewGameDlg() as you suggested and really use it once we add description or find another way how to emphasize how computer plays. 
This way user experience will remain the same with just player's type changed (Default Weak/Normal/Hard).
Also, opposed to the dead code, I would like to keep code in example dir because it shows what is really needed in order to add new type of ai player to the game.  </pre>
<br />




<p>- Nemanja</p>


<br />
<p>On October 8th, 2013, 1:03 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. 8, 2013, 1:03 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;">The default AI is handled by players/ai/default/default.cpp. The difficulty is wrapped in own classes so that the external interface is the same for all players (AI and human).

Adding new AI players is as easy as to implement the code in an own class, add a GUI wrapper (see ai/example) and add the GUI wrapper to m_selectablePlayer in NewGameDlg constructor.</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">(0e8a354)</span></li>

 <li>dialogs/newgamedlg.h <span style="color: grey">(10ce4ee)</span></li>

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

</ul>

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







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








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