D21478: Fix passing local file paths on the command line

Fabian Vogt noreply at phabricator.kde.org
Wed May 29 14:01:17 BST 2019


fvogt added inline comments.

INLINE COMMENTS

> App.cpp:280
>          {
> -            QUrl url(  m_args->positionalArguments().at( i ) );
> +            QUrl url( QUrl::fromUserInput( m_args->positionalArguments().at( i ), QString(), QUrl::AssumeLocalFile ) );
>              //TODO:PORTME

Did it previously also handle relative paths such as `./foo.ogg`? If so, passing `QDir::currentPath()` instead of a null string is AFAICT necessary.

REPOSITORY
  R181 Amarok

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

To: wbauer, #amarok
Cc: fvogt, amarok-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok-devel/attachments/20190529/215b8802/attachment-0001.html>


More information about the Amarok-devel mailing list