carewolf added inline comments.

INLINE COMMENTS

> alexeymin wrote in regexpcache.cpp:81
> can probably just be `QString suffixMatch(QLatin1String("^.*\\.("));` then, 
> without `=`

Sure, but it hardly matters, copy-elision will make the statements equivalent.

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