> On July 10, 2011, 1:20 p.m., Ingo Klöcker wrote: > > This changes the behavior of toLocalFile() for non-local URLs. Instead of > > an empty string your version returns the (remote) path. I'm not sure how > > relevant this is because it makes little sense to call toLocalFile() on > > non-local URLs. OTOH, some developers might check the return value to > > toLocalFile() instead of using isLocalFile(). > > > > I suggest getting your fix into Qt instead of adding a workaround to > > kdelibs.
That is true, but the patch can be easily change to accomodate that case as well. I have opened a ticket upstream, but just in case it is not fixed there at least for Qt 4.8 I have also updated the patch for KUrl not to break previous behavior for non-local URLs. - Dawit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101906/#review4572 ----------------------------------------------------------- On July 10, 2011, 3:34 p.m., Dawit Alemayehu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/101906/ > ----------------------------------------------------------- > > (Updated July 10, 2011, 3:34 p.m.) > > > Review request for kdelibs and David Faure. > > > Summary > ------- > > The attached patch changes KUrl::toLocalFile to use QUrl::path() when > extracting the path from the current URL on non-Windows platforms so that > KUrl would return the correct path when KUrl::toLocalFile is called on an > absolute path url whose top level contains a ':', e.g. "file:///A:/". > > > This addresses bug 194746. > http://bugs.kde.org/show_bug.cgi?id=194746 > > > Diffs > ----- > > kdecore/io/kurl.cpp acfc9a1 > > Diff: http://git.reviewboard.kde.org/r/101906/diff > > > Testing > ------- > > > Thanks, > > Dawit > >
