KURL::fromPathOrURL

[David Faure]

On Tuesday 03 January 2006 02:43, Aaron J. Seigo wrote:
> Index: kdecore/kurl.cpp
> ===================================================================
> --- ../../kdecore/kurl.cpp      (revision 492760)
> +++ ../../kdecore/kurl.cpp      (working copy)
> @@ -2369,7 +2369,7 @@
>      if ( !text.isEmpty() )
>      {
>          if (!QDir::isRelativePath(text))
> -            url.setPath( text );
> +            url.setPath( KShell::tildeExpand( text ) );
>          else
>              url = text;
>      }

Did you test it?
!isRelativePath() means "it starts with '/'", so if it starts with a '~' you won't get into this part of the if.

Other than that I'm ok with the idea of the patch.

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).