poboiko added inline comments.
INLINE COMMENTS
> regexpcache.cpp:60
> + }
> f.replace(QLatin1Char('.'), QStringLiteral("\\."));
> f.replace(QLatin1Char('?'), QLatin1Char('.'));
A quick note for future. There is
`QRegularExpression::wildcardToRegularExpression()`, which apparently does
precicely this.
It is there since Qt 5.12 though, while we depend on Qt 5.10. But Plasma
already bumped dependency to 5.12, so so we can change it when Frameworks will
follow this bump.
REPOSITORY
R293 Baloo
BRANCH
submit_unindexed
REVISION DETAIL
https://phabricator.kde.org/D21579
To: bruns, #baloo, ngraham, astippich, poboiko, broulik
Cc: kde-frameworks-devel, LeGast00n, domson, ashaposhnikov, michaelh,
astippich, spoorun, ngraham, bruns, abrahams