<table><tr><td style="">mdawson 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/D4604" rel="noreferrer">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/D4604#inline-18873" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">dfaure</span> wrote in <span style="color: #4b4d51; font-weight: bold;">kconfigbackend.h:210</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">In general I agree that leaving dead code is bad.</p>

<p style="padding: 0; margin: 8px;">In this particular case, though, the other side of the plugin code is still there<br />
("#if 0 <em> TODO port to Qt5 plugin loading" in kconfigbackend.cpp), and actually this is the bit that *was* ported to Qt5 plugin loading, so it will be useful if anyone decides to finally implement plugin based backends. Maybe I can make it<br />
#if 0 </em> TODO re-enable if the plugin loading code is re-enabled<br />
instead of just "//" which is indeed uglier.</p>

<p style="padding: 0; margin: 8px;">Alternatively, we decide that plugin-based backends are not even on the far-future radar and we clean up both sides completely. Works for me too. The whole idea was crazy anyway (apps make assumptions about config files being files, if one day we change that for e.g. something like gconf then it won't be as transparent as just switching to a different backend).</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">Sure, #if 0 is best since it has the code, and they will at least match.</p>

<p style="padding: 0; margin: 8px;">I'd actually like to try and bring this back sometime, but as you say it would be very difficult.  I don't know if it will ever come to pass, so I don't want to hold up actual improvements for a nice to have.  I definitely don't know if it would ever be plugin based, or just built into KConfig.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R237 KConfig</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D4604" rel="noreferrer">https://phabricator.kde.org/D4604</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>dfaure, mdawson<br /><strong>Cc: </strong>Frameworks<br /></div>