D8569: Fixed launch url composition from launch configuration server, path and arguments

[Alexander Zhigalin]

zhigalin added inline comments.

INLINE COMMENTS

> apol wrote in executebrowserplugin.cpp:87
> Probably not. Now if we need to go this way, it would probably make sense to use a different configuration setting, otherwise people will get weird URLs.

What about

  if( !host.contains(QStringLiteral("://")) )
  {
      host.prepend(QStringLiteral("http://"));
  }

REPOSITORY
  R67 KDevelop Execute Browser

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

To: santilin
Cc: zhigalin, apol, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20171103/8c0b92d5/attachment.html>