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





 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Reached the point i can't review anymore since my knowledge of ksquares is close to 0, I'll let Matt or anyone else continue the "real" review.</pre>
 <br />







<p>- Albert</p>


<br />
<p>On April 29th, 2012, 4:04 p.m., Tom Vincent Peters wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.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 and Matt Willliams.</div>
<div>By Tom Vincent Peters.</div>


<p style="color: grey;"><i>Updated April 29, 2012, 4:04 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;">Hard AI for KSquares
Added per player AI difficulty  
Moved AI settings from normal game to demo mode
Added player count parameter for demo mode

known unintentional changes to other AI:
medium AI probably will do hard-hearted handouts in most cases</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;">100 autoplay games medium against hard
a lot of games against hard ai</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>/trunk/KDE/kdegames/ksquares/ChangeLog <span style="color: grey">(1292133)</span></li>

 <li>/trunk/KDE/kdegames/ksquares/src/aicontroller.h <span style="color: grey">(1292133)</span></li>

 <li>/trunk/KDE/kdegames/ksquares/src/aicontroller.cpp <span style="color: grey">(1292133)</span></li>

 <li>/trunk/KDE/kdegames/ksquares/src/gameboardscene.cpp <span style="color: grey">(1292133)</span></li>

 <li>/trunk/KDE/kdegames/ksquares/src/ksquares.kcfg <span style="color: grey">(1292133)</span></li>

 <li>/trunk/KDE/kdegames/ksquares/src/ksquaresdemowindow.h <span style="color: grey">(1292133)</span></li>

 <li>/trunk/KDE/kdegames/ksquares/src/ksquaresdemowindow.cpp <span style="color: grey">(1292133)</span></li>

 <li>/trunk/KDE/kdegames/ksquares/src/ksquareswindow.cpp <span style="color: grey">(1292133)</span></li>

 <li>/trunk/KDE/kdegames/ksquares/src/main.cpp <span style="color: grey">(1292133)</span></li>

 <li>/trunk/KDE/kdegames/ksquares/src/newgamewidget.ui <span style="color: grey">(1292133)</span></li>

 <li>/trunk/KDE/kdegames/ksquares/src/prefs_ai.ui <span style="color: grey">(1292133)</span></li>

 <li>/trunk/KDE/kdegames/ksquares/src/settings.kcfgc <span style="color: grey">(1292133)</span></li>

</ul>

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




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








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