<table><tr><td style="">mwolff accepted this revision.<br />mwolff added a comment.<br />This revision is now accepted and ready to land.
</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/D3040" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>remove the moc, then feel free to push</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/D3040#inline-20927" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">igorkushnir</span> wrote in <span style="color: #4b4d51; font-weight: bold;">CMakeLists.txt:7</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">It's related. You have already written this comment, I've replied that I had changed <tt style="background: #ebebeb; font-size: 13px;">StandardDocumentationView</tt> ABI, then you have agreed to leave this change in.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">sorry :-/</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/D3040#inline-20928" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">igorkushnir</span> wrote in <span style="color: #4b4d51; font-weight: bold;">zoomcontroller.h:112</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">This is a <tt style="background: #ebebeb; font-size: 13px;">ZoomController</tt> class. It has a zoom factor, which is referred to as simply "factor" in member function names. It's obvious what factor it is as ZoomController shouldn't have any other factors. Do you want me to rename all 3 members: <tt style="background: #ebebeb; font-size: 13px;">factor()</tt>, <tt style="background: #ebebeb; font-size: 13px;">setFactor()</tt> and <tt style="background: #ebebeb; font-size: 13px;">factorChanged()</tt>? What about <tt style="background: #ebebeb; font-size: 13px;">d->m_factor</tt> and other variables in zoomcontroller.cpp?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">leave it as is then, you are right - doesn't make much of a difference</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R33 KDevPlatform</div></div></div><br /><div><strong>BRANCH</strong><div><div>documentation-zoom-new-trimmed</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3040" rel="noreferrer">https://phabricator.kde.org/D3040</a></div></div><br /><div><strong>To: </strong>igorkushnir, KDevelop, mwolff<br /><strong>Cc: </strong>rjvbb, mwolff, apol, kdevelop-devel<br /></div>