alexeymin added inline comments.

INLINE COMMENTS

> regexpcache.cpp:81
>      // Combine all suffixes into one large RE: "^.*(foo|bar|baz)$"
> -    auto suffixMatch = QLatin1String("^.*\\.(");
> +    QString suffixMatch = QLatin1String("^.*\\.(");
>      suffixMatch += suffixes.join(QChar('|'));

can probably just be `QString suffixMatch(QLatin1String("^.*\\.("));` then, 
without `=`

REPOSITORY
  R293 Baloo

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

To: carewolf, bruns, ngraham
Cc: alexeymin, kde-frameworks-devel, #baloo, LeGast00n, domson, ashaposhnikov, 
michaelh, astippich, spoorun, ngraham, bruns, abrahams

Reply via email to