Re: [PATCH 1/2] Fix URL parsing
- Date: Sat, 28 May 2016 15:37:53 +0200
- From: Arno <arno@xxxxxxxxxxxxx>
- Subject: Re: [PATCH 1/2] Fix URL parsing
On 5/28/2016 3:01 PM, Thiago Macieira wrote:
> Em sábado, 28 de maio de 2016, às 01:08:24 BRT, Arno Moeller escreveu:
>> As it seems, QUrl does not know about rdp://, so url.scheme() is empty.
>> The old code tried to "fix" it by replacing the scheme with vnc://,
>> because, as the comment stated, it was default in kde3.
> That doesn't make sense. QUrl does know about *any* schemes except "file",
> which means it recognises them all.
Hmm, thanks for the hint. krdc calls
QURL::fromLocalFile("rdp://i.p.a.ddress") which is wrong.
rpd://<something> is clearly not a local file, but since krdc said it
it, we end up with file://rdp://i.p.a.ddress, which is garbage...
I kinda wondered why it worked when setting the URL in the constructor.
I wonder how this is supposed to work...