<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;">You're style does not follow the coding style of the file at all, please have a look at the style of the file and try to follow it, mainly you are missing spaces around assignments, comparisons, etc</pre>
 <br />







<p>- Albert</p>


<br />
<p>On April 28th, 2012, 12:24 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 28, 2012, 12:24 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>