----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/101906/#review4572 -----------------------------------------------------------
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. - Ingo On July 10, 2011, 7:09 a.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, 7:09 a.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 > >