dfaure planned changes to this revision.
dfaure added inline comments.

INLINE COMMENTS

> kurifilter.cpp:665
> +    QStringList res;
> +    res.reserve(d->pluginList.size());
> +    std::transform(d->pluginList.constBegin(), d->pluginList.constEnd(), 
> res.begin(), [](const KUriFilterPlugin *plugin) { return 
> plugin->objectName(); });

Interesting, to review my own commit from a month ago, after I forgot its 
details...

Dude, this code crashes, reserve+transform needs back_inserter, otherwise use 
resize!

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D9100

To: dfaure, apol, davidedmundson, arichardson, bshah
Cc: #frameworks

Reply via email to