<table><tr><td style="">fabiank added inline comments.
</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/D11814">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D11814#inline-59216">View Inline</a><span style="color: #4b4d51; font-weight: bold;">aacid</span> wrote in <span style="color: #4b4d51; font-weight: bold;">kcardscene.cpp:1253</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">This seems like something you would want to keep? (if it compiles)</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Well, it doesn't, because multRectSize doesn't exist anymore since <a href="https://phabricator.kde.org/R410:efb0f40b7b16fb70d32cecd91dd8d4d9decd61c5" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">efb0f40b7b16fb70d32cecd91dd8d4d9decd61c5</a>. Though, restoring that function (and wrapping it in #if DEBUG_LAYOUT) is of course also an alternative.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D11814#inline-59217">View Inline</a><span style="color: #4b4d51; font-weight: bold;">aacid</span> wrote in <span style="color: #4b4d51; font-weight: bold;">idiotsolver.cpp:299</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Are you really sure about this? This changes the assert totally</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Indeed it does, and for the better I'd say ;-). Before it was an assignment to Wp[i], if and only if asserts were active. The only reason why this assignment didn't cause any issues was that lhs and rhs of the assignment should be the same, as we assert now.</p>

<p style="padding: 0; margin: 8px;">Wp[i] == &W[i][Wlen[i] - 1] is actually an invariant through much of the code base, except when it is temporarily broken when moving cards around (and replacing Wp, Wlen, W and related variables is one of the refactorings I've planned at a later point).</p></div></div></div></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/D11814">https://phabricator.kde.org/D11814</a></div></div><br /><div><strong>To: </strong>fabiank, KDE Games<br /><strong>Cc: </strong>aacid, KDE Games<br /></div>