apol accepted this revision.
apol added a comment.
This revision is now accepted and ready to land.


  LGTM otherwise

INLINE COMMENTS

> KNSBackend.cpp:149
> +        const QString catName = cat->name().append(QLatin1Char('/'));
> +        for (auto& potentialSubCat: categories) {
> +            if(potentialSubCat->name().startsWith(catName)) {

const auto&

REPOSITORY
  R134 Discover Software Store

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

To: leinir, #discover_software_store, apol, ngraham
Cc: plasma-devel, #discover_software_store, jraleigh, bam, GB_2, ragreen, 
ixoos, Pitel, ZrenBot, James, lesliezhai, ali-mohamed, jensreuterberg, abetts, 
sebas, apol, mart

Reply via email to