D13189: [kcoredirlister] Remove as many url.toString() as possible

Aleix Pol Gonzalez noreply at phabricator.kde.org
Tue May 29 10:56:54 UTC 2018


apol added a comment.


  +1 makes sense

INLINE COMMENTS

> kcoredirlister.cpp:547
>  
> -    QUrl url(_url);
> -    url = url.adjusted(QUrl::StripTrailingSlash);
> -    const QString urlStr = url.toString();
> +    QUrl url = _url.adjusted(QUrl::StripTrailingSlash);
>  

const?

REPOSITORY
  R241 KIO

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

To: jtamate, dfaure, #frameworks
Cc: apol, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180529/db9ca698/attachment.html>


More information about the Kde-frameworks-devel mailing list