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