<table><tr><td style="">markg accepted this revision.<br />markg 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/D8347" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D8347#156930" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D8347#156930</a>, <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@ngraham</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I still need to add a <span class="phabricator-remarkup-mention-unknown">@since</span> in the header documentation, after which point hopefully <a href="https://phabricator.kde.org/p/markg/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@markg</a> will approve. :)</p></div>
</blockquote>

<p>+1 then. I trust you do that.<br />
Also note that a proper container for what you're doing is two sets (default plugins and the blacklisted ones) and calling the std::set_difference algorithm on them; see: <a href="https://www.fluentcpp.com/2017/01/09/know-your-algorithms-algos-on-sets/" class="remarkup-link" target="_blank" rel="noreferrer">https://www.fluentcpp.com/2017/01/09/know-your-algorithms-algos-on-sets/</a><br />
That's just minor details though, feel free to ignore that :)</p>

<p>I remain skeptical of your entire goal here though. It's good and well intended, that's for sure. But ultimately this is for Dolphin and the previews which in turn means that by default all but one plugin is going to be enabled. I'm skeptical about that because there is always an issue with doing that. Either bad plugins causing weird behavior, bad data (just plain and simple broken data) causing weird behavior. And probably a gazillion other reasons once it is enabled and used in the next dolphin version...</p>

<p>I hope you're up for a lot of bug reports :D</p>

<p>On the other hand, it would be very cool if this starts working properly!</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8347" rel="noreferrer">https://phabricator.kde.org/D8347</a></div></div><br /><div><strong>To: </strong>ngraham, Frameworks, broulik, Dolphin, markg<br /><strong>Cc: </strong>markg, anthonyfieroni, elvisangelaccio, Frameworks<br /></div>