<table><tr><td style="">mreeves added a comment.
</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/D20892">View Revision</a></tr></table><br /><div><div><p>Q_DECL_OVERRIDE is a nop on compilers that don't support override. So this will make the functions non virtual in that case. The original revision did not have this problem. Please test this on such a compiler.</p></div></div><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/D20892#inline-117302">View Inline</a><span style="color: #4b4d51; font-weight: bold;">mlaurent</span> wrote in <span style="color: #4b4d51; font-weight: bold;">abstract_fc_solve_solver.h:28</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">remove virtual keyword.<br />
Q_DECL_OVERRIDE is enough. And I think that we can use override directly.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Use overide directly or these functions will be none virtual on compilers that don't support overide.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R410 KPatience</div></div></div><br /><div><strong>BRANCH</strong><div><div>elim-gcc-warnings</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20892">https://phabricator.kde.org/D20892</a></div></div><br /><div><strong>To: </strong>shlomif, KDE Games, mlaurent<br /><strong>Cc: </strong>mreeves, mlaurent, kde-games-devel<br /></div>