From kde-commits  Thu Dec 04 11:06:47 2008
From: Carlo <brandon.ml () gmail ! com>
Date: Thu, 04 Dec 2008 11:06:47 +0000
To: kde-commits
Subject: Re: KDE/kdelibs/kdecore/io
Message-Id: <3262b6180812040306v15dcb099sf1b9702d4f6ade7 () mail ! gmail ! com>
X-MARC-Message: https://marc.info/?l=kde-commits&m=122838885911714

On Thu, Dec 4, 2008 at 10:59 AM, Andreas Pakulat <apaku@gmx.de> wrote:
> On 04.12.08 10:30:10, Carlo wrote:
>> On Thu, Dec 4, 2008 at 9:02 AM, Andreas Pakulat <apaku@gmx.de> wrote:
>> > On 03.12.08 23:58:01, Carlo Segato wrote:
>> >> SVN commit 892299 by segato:
>> >>
>> >> convert backslashes to forwardslashes in cleanpath, otherwise it returns an empty string
>> >
>> > Are you sure this is correct? If I understand this correctly I can't have a
>> > url that uses forward-slash as path separator, but contains backslashes
>> > anymore. Apart from that this looks like something thats win32 specific and
>> > might rather show a bug in some other code. I don't think you're supposed
>> > to use native paths with KUrl, as a Url is defined to always use forward
>> > slashes as path separator.
>> >
>> > I think this needs to be reverted.
>> >
>> > Andreas
>> >
>> > --
>> > Make a wish, it might come true.
>> >
>> yes it's win32 specific, the problem is that if I open a file from
>> command line with kate like this "kate c:\file.txt" kate uses
>> cleanpath that returns c:/ since it can't find any forwardslashes, I
>> didn't tought about path that contains \ as non dir separator, and I
>> can't think of a way to change only separators
>
> Well, the error is in kate, it apparently doesn't use QFile&Co to convert
> the path read from argv. It needs to do that, so the path it hands over to
> KUrl or anything else is converted by Qt from native path to
> forward-slashes - properly.
>
> So please revert and fix kate instead.
>
> Andreas
>
> --
> You'll wish that you had done some of the hard things when they were easier
> to do.
>

what do you mean that it should use qfile?
anyway I was looking at cleanPath in QDir and it does this

QChar dir_separator = separator();
    if(dir_separator != QLatin1Char('/'))
	name.replace(dir_separator, QLatin1Char('/'));

we can do the same in kurl or even use #ifdef so only windows will be affected