<table><tr><td style="">michaelh accepted this revision.<br />michaelh 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/D11279">View Revision</a></tr></table><br /><div><div><p>Ha! Wonderful, this touches a question I raised on kde-devel ML recently. Say we wanted to test this patch. We would need some kind of mock drive, right? I'd very much appreciate if you would lay out your thought on this.<br />
Secondly this illustrates that my comprehension of ABI is still fragmentary. Obviously this is not breaking, but I thought it would.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Dolphin does not gray out sort options and additional roles for Baloo roles when disabled but that is a bug unrelated to this change</p></blockquote>

<p>Didn't see that.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R293 Baloo</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11279">https://phabricator.kde.org/D11279</a></div></div><br /><div><strong>To: </strong>broulik, michaelh<br /><strong>Cc: </strong>Frameworks, ashaposhnikov, michaelh, spoorun, nicolasfella, alexeymin<br /></div>