David Faure said the following, On 2008-07-10 13:27: > On Thursday 10 July 2008, Jaroslaw Staniek wrote: >> (for review) >> Native "Open With" dialog on Windows (by default). >> This allows to use (and associate) given file extensions with any Windows >> application directly, as users expect. >> >> - moved KRun::KRunPrivate declaration to krun_p.h >> - added KRun::KRunPrivate::displayNativeOpenWithDialog() for Windows only >> in krun_win.cpp >> - added a note to KOpenWithDialog that using KRun::displayOpenWithDialog() >> is recommended > > Why should this be in KRun? I think this is an odd place for the main displayOpenWithDialog() call. > Can't it be a static method of KOpenWithDialog instead, just like we do for KFileDialog? I wanted to ask this question, and have the same proposal, as I dislike the location too... But it's in kdelibs already. I can make displayOpenWithDialog() obsolete and add KOpenWithDialog::openWith() or so. -- regards / pozdrawiam, Jaroslaw Staniek Sponsored by OpenOffice Polska (http://www.openoffice.com.pl/en) to work on Kexi & KOffice (http://www.kexi.pl/en, http://www.koffice.org/kexi) KDE Libraries for MS Windows (http://windows.kde.org)