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



 <p>Ship it!</p>



 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Tested and it works as intended. Probably fix these minor issues and then commit.</pre>
 <br />







<div>




<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="https://git.reviewboard.kde.org/r/116660/diff/1/?file=252954#file252954line27" style="color: black; font-weight: bold; text-decoration: underline;">src/configure-practice/thresholdoptions.cpp</a>
    <span style="font-weight: normal;">

     (Diff revision 1)

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



 
 

 <tbody>

  <tr>
    <th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">27</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="cp">#include "practice/sessionmanager.h"</span></pre></td>
    <th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">27</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="c1"><span class="hl">//</span>#include "practice/sessionmanager<span class="hl">base</span>.h"</span></pre></td>
  </tr>

 </tbody>

</table>

<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">delete</pre>
</div>
<br />

<div>




<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="https://git.reviewboard.kde.org/r/116660/diff/1/?file=252955#file252955line191" style="color: black; font-weight: bold; text-decoration: underline;">src/parleymainwindow.h</a>
    <span style="font-weight: normal;">

     (Diff revision 1)

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

 <tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
  <tr>

   <td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">private:</pre></td>

  </tr>
 </tbody>



 
 

 <tbody>

  <tr>
    <th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">190</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="n">Practice</span><span class="o">::</span><span class="n">SessionManager</span> <span class="n">m_sessionManager</span><span class="p">;</span></pre></td>
    <th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">191</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="c1"><span class="hl">//</span>Practice::SessionManager<span class="hl">Continuous</span> m_sessionManager;</span></pre></td>
  </tr>

 </tbody>

</table>

<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">needed? If not, delete.</pre>
</div>
<br />

<div>




<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="https://git.reviewboard.kde.org/r/116660/diff/1/?file=252981#file252981line44" style="color: black; font-weight: bold; text-decoration: underline;">src/practice/sessionmanagerfixed.cpp</a>
    <span style="font-weight: normal;">

     (Diff revision 1)

    </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">44</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="c">    const int MaxNewWords = 5;      // FIXME: Make this a config option.<span class="ew"> </span></span></pre></td>
  </tr>

 </tbody>

</table>

<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Trailing space</pre>
</div>
<br />

<div>




<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="https://git.reviewboard.kde.org/r/116660/diff/1/?file=252981#file252981line56" style="color: black; font-weight: bold; text-decoration: underline;">src/practice/sessionmanagerfixed.cpp</a>
    <span style="font-weight: normal;">

     (Diff revision 1)

    </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">56</font></th>
    <td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">    <span class="k">while</span> <span class="p">(</span><span class="n">i</span> <span class="o"><</span> <span class="n">m_allTestEntries</span><span class="p">.</span><span class="n">count</span><span class="p">()</span><span class="ew"> </span></pre></td>
  </tr>

 </tbody>

</table>

<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Trailing space</pre>
</div>
<br />

<div>




<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="https://git.reviewboard.kde.org/r/116660/diff/1/?file=252984#file252984line22" style="color: black; font-weight: bold; text-decoration: underline;">src/practice/writtenpracticevalidator.h</a>
    <span style="font-weight: normal;">

     (Diff revision 1)

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



 
 

 <tbody>

  <tr>
    <th bgcolor="#e9eaa8" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">22</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="cp">#include "sessionmanager.h"</span></pre></td>
    <th bgcolor="#e9eaa8" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">22</font></th>
    <td bgcolor="#fdfebc" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "><span class="c1"><span class="hl">//</span>#include "sessionmanager<span class="hl">base</span>.h"</span></pre></td>
  </tr>

 </tbody>

</table>

<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">delete if not needed</pre>
</div>
<br />



<p>- Amarvir Singh</p>


<br />
<p>On March 8th, 2014, 10:14 p.m. IST, Inge Wallin wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://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 Edu and Amarvir Singh.</div>
<div>By Inge Wallin.</div>


<p style="color: grey;"><i>Updated March 8, 2014, 10:14 p.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
parley
</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;">This patch implement fixed sessions for Parley.  With that I mean that training is split up into sessions with a fixed number maximum number of words. The default value for the number is 20 but it could also be fewer. At most 5 of those can be new words, so it also makes it easier to learn new vocabulary by not allowing an unwieldy number of new words at a time. Before this patch a session was all the words that were due based on time since last practice and grade, which meant that a session could potentially consist of thousands of words.

Technically this patch works by splitting the old SessionManager into a base class (SessionManagerBase) and a specific class (SessionManagerContinuous) and then implementing a new class SessionManagerFixed.  The SessionManager was recently renamed into that name from the old TestEntryManager which didn't describe what it did at all.

This is the first milestone in the project to create a new session system.  Step 2 will be with adding some configuration and step 3 will be a new graphic look.

Note that in this version of the patch the numbers 20 (max size) and 5 (max new words) are replaced with 7 and 3 to make testing easier.</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 with a special version of a word list I am working with.  I will attach it here.</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>src/settings/languagepropertiespage.cpp <span style="color: grey">(e324167)</span></li>

 <li>src/practice/writtenpracticevalidator.h <span style="color: grey">(d38190e)</span></li>

 <li>src/practice/writtenbackendmode.cpp <span style="color: grey">(4cb723e)</span></li>

 <li>src/practice/sessionmanagerfixed.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/practice/writtenbackendmode.h <span style="color: grey">(dd8d3ee)</span></li>

 <li>src/practice/sessionmanagerfixed.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/practice/sessionmanagercontinuous.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/practice/sessionmanagercontinuous.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/practice/sessionmanagerbase.cpp <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/practice/sessionmanagerbase.h <span style="color: grey">(PRE-CREATION)</span></li>

 <li>src/practice/sessionmanager.cpp <span style="color: grey">(adea30b)</span></li>

 <li>TODO-SESSIONS <span style="color: grey">(PRE-CREATION)</span></li>

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

 <li>src/configure-practice/blockoptions.h <span style="color: grey">(1882994)</span></li>

 <li>src/configure-practice/thresholdoptions.cpp <span style="color: grey">(1fb55e1)</span></li>

 <li>src/parleymainwindow.h <span style="color: grey">(318312c)</span></li>

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

 <li>src/practice/DESIGN <span style="color: grey">(99d4c0b)</span></li>

 <li>src/practice/comparisonbackendmode.h <span style="color: grey">(e5cb5d0)</span></li>

 <li>src/practice/comparisonbackendmode.cpp <span style="color: grey">(a3efac8)</span></li>

 <li>src/practice/conjugationbackendmode.h <span style="color: grey">(8cc59ee)</span></li>

 <li>src/practice/conjugationbackendmode.cpp <span style="color: grey">(fa365d9)</span></li>

 <li>src/practice/examplesentencebackendmode.h <span style="color: grey">(38941e1)</span></li>

 <li>src/practice/examplesentencebackendmode.cpp <span style="color: grey">(0cc812c)</span></li>

 <li>src/practice/genderbackendmode.h <span style="color: grey">(9a4a28a)</span></li>

 <li>src/practice/genderbackendmode.cpp <span style="color: grey">(b2cc7c1)</span></li>

 <li>src/practice/multiplechoicebackendmode.h <span style="color: grey">(a77265e)</span></li>

 <li>src/practice/multiplechoicebackendmode.cpp <span style="color: grey">(51f35a3)</span></li>

 <li>src/practice/practicemainwindow.h <span style="color: grey">(93fcd66)</span></li>

 <li>src/practice/practicemainwindow.cpp <span style="color: grey">(6fe7aa1)</span></li>

 <li>src/practice/practicestatemachine.h <span style="color: grey">(81cff91)</span></li>

 <li>src/practice/practicestatemachine.cpp <span style="color: grey">(16f86c1)</span></li>

 <li>src/practice/practicesummarycomponent.h <span style="color: grey">(41cc83b)</span></li>

 <li>src/practice/practicesummarycomponent.cpp <span style="color: grey">(bffb584)</span></li>

 <li>src/practice/sessionmanager.h <span style="color: grey">(64e70c9)</span></li>

</ul>

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



<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>

<ul>

 <li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2014/03/08/f5213df1-cfe8-44cc-9d16-c2956bf39076__Thai-numbers-copy.kvtml">A small test file</a></li>

</ul>





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








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