[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: [patch] KFileWidget: fix changing the current dir using url combo
From:       David Faure <faure () kde ! org>
Date:       2008-04-11 21:31:52
Message-ID: 200804112331.53204.faure () kde ! org
[Download RAW message or body]

On Friday 11 April 2008, Jarosław Staniek wrote:
> 
> for review:
> 
> Fix changing the current dir using the 'recent url' combo box.
> Upon accepting new url in the 'directory' url combo box, append '/' if needed: 
> the combo does not add it, but tokenize() expects it because uses 
> KUrl::setFileName(), which would silently remove the last segment of the path.

So it should probably use addPath() instead of setFileName()?

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

[prev in list] [next in list] [prev in thread] [next in thread]