From kde-core-devel  Sat Dec 23 10:09:34 2006
From: Ralf Habacker <ralf.habacker () freenet ! de>
Date: Sat, 23 Dec 2006 10:09:34 +0000
To: kde-core-devel
Subject: Re: KUrl(QString) fix
Message-Id: <458D005E.7020109 () freenet ! de>
X-MARC-Message: https://marc.info/?l=kde-core-devel&m=116688600125529

David Faure schrieb:
>
> So? We haven't released kde-4.0 yet, it can be made non-inline any Monday.
> It is inline right now only because of the overload that takes an argument, but the intent
> wasn't that KUrl::path would work any differently from QUrl::path. But since we already
> disagree with QUrl about a few things (like the use of toString), I'm not actually 
> opposed to a solution where path() would work differently.
>   
With the increased knowledge of this stuff I think that there is no 
reason to change path() behavior because we can use toLocalFile().
More important would be to have a toLocalFile() method with the 
'AdjustPathOption trailing' parameter as path() have

  QString path( ) const;
  QString path( AdjustPathOption trailing ) const;

->

  QString toLocalFile() const;
  QString toLocalFile(AdjustPathOption trailing ) const;


Ralf