<table><tr><td style="">huftis requested changes to this revision.<br />huftis added a comment.<br />This revision now requires changes to proceed.
</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/D16177">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D16177#342429" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D16177#342429</a>, <a href="https://phabricator.kde.org/p/shlomif/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@shlomif</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>Sorry - this should be applied against <a href="https://phabricator.kde.org/D16176" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D16176</a> .</p></div>
</blockquote>

<p>Thank you. I can confirm that this fixes the crashing bug in <a href="https://phabricator.kde.org/D16176" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/D16176</a>.</p>

<p>With the fix, when the ‘demo mode’ runs out of valid moves to make, the demo mode seems to be correctly ended, i.e. the ‘Demo’ icon on the toolbar is deselected and changed back to a ‘Play’ icon. The only problem I can see is that (for the game in <a href="https://phabricator.kde.org/D16176" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D16176</a>) the status line nows says <tt style="background: #ebebeb; font-size: 13px;">Solver: Calculating...</tt> after finishing the demo. It should be empty (as it is when actually running the demo), since it’s not actually doing any calculations. This should be fixed before commiting.</p></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/D16177">https://phabricator.kde.org/D16177</a></div></div><br /><div><strong>To: </strong>shlomif, KDE Games, huftis<br /><strong>Cc: </strong>huftis, kde-games-devel<br /></div>