D9333: Make sure that old URLs do not appear duplicated in places model

Elvis Angelaccio noreply at phabricator.kde.org
Sun Jan 21 14:28:15 GMT 2018


elvisangelaccio added inline comments.

INLINE COMMENTS

> placesitemmodel.cpp:58
>      // added, which is only read by PlacesItemModel.
> -    const char AppNamePrefix[] = "-places-panel";
> -
> +    const char AppNamePrefix[] = QStringLiteral("-places-panel");
>      static QList<QUrl> balooURLs = {

hmm why not `const QString AppNamePrefix = QStringLiteral("-places-panel");` ?

REPOSITORY
  R318 Dolphin

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

To: renatoo, mwolff, #dolphin, ngraham, elvisangelaccio
Cc: elvisangelaccio, broulik, ervin, ngraham, #dolphin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20180121/76d40b45/attachment.htm>


More information about the kfm-devel mailing list