<table><tr><td style="">fabiank created this revision.<br />fabiank added reviewers: aacid, shlomif, yurchor, piotrkakol.<br />Herald added a reviewer: KDE Games.<br />Herald added a subscriber: kde-games-devel.<br />fabiank requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D27432">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Instead of changing the logic in dealer.cpp as in <a href="https://phabricator.kde.org/D27336" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D27336</a>, only change the moveCount function for FreeCell.</p>

<p>Pro:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">less invasive change, less likely to cause regressions in other places</li>
</ul>

<p>Contra:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">This might only pamper over an actual issue, which <a href="https://phabricator.kde.org/D27336" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D27336</a> would fix.</li>
</ul></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Started a new Freecell game (without multiple aces), and checked that the count is sound</li>
<li class="remarkup-list-item">Loaded existing Freecell games, and verified that the count is shown as expected</li>
<li class="remarkup-list-item">Started Freecell game where the initial state had two aces; verified that after the auto move completed, the displayed count was 2 (and not 1)</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R410 KPatience</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27432">https://phabricator.kde.org/D27432</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>dealer.h<br />
freecell.cpp<br />
freecell.h</div></div></div><br /><div><strong>To: </strong>fabiank, aacid, shlomif, yurchor, piotrkakol, KDE Games<br /><strong>Cc: </strong>kde-games-devel<br /></div>