D23898: [KUrlNavigatorButton] Fix QString usage to not use [] out of bounds.

Elvis Angelaccio noreply at phabricator.kde.org
Thu Sep 12 13:19:34 BST 2019


elvisangelaccio accepted this revision.
elvisangelaccio added a comment.
This revision is now accepted and ready to land.


  I don't have 5.14 around atm, so I trust you've tested this :)

INLINE COMMENTS

> kurlnavigatortest.cpp:265
> +    KFilePlacesModel model;
> +    const QString pwd = QDir::currentPath();
> +    const QUrl url = QUrl::fromLocalFile(pwd);

Did you mean `cwd` ?

REPOSITORY
  R241 KIO

BRANCH
  master

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

To: dfaure, elvisangelaccio
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190912/f019515e/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list