D21604: Fix building with Qt 5.13

Alexey Min noreply at phabricator.kde.org
Wed Jun 5 16:08:38 BST 2019


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190605/0def8e74/attachment.html>


More information about the Kde-frameworks-devel mailing list