----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/117078/#review54455 -----------------------------------------------------------
Thanks for this work! src/core/kioglobal_p.h <https://git.reviewboard.kde.org/r/117078/#comment38097> How is this different from QFileInfo::symLinkTarget()? Why not just port to that? src/core/kioglobal_p_win.cpp <https://git.reviewboard.kde.org/r/117078/#comment38100> Bonus points for contributing a static method to QProcess... src/core/kioglobal_p_win.cpp <https://git.reviewboard.kde.org/r/117078/#comment38098> Bonus points for contributing this to Qt... src/core/kioglobal_p_win.cpp <https://git.reviewboard.kde.org/r/117078/#comment38099> Why not QFileInfo::isSymLink()? src/widgets/kpropertiesdialog.cpp <https://git.reviewboard.kde.org/r/117078/#comment38101> Or just kcom->setItems(userNames), no loop needed; src/widgets/kpropertiesdialog.cpp <https://git.reviewboard.kde.org/r/117078/#comment38102> (same here) src/widgets/kpropertiesdialog.cpp <https://git.reviewboard.kde.org/r/117078/#comment38103> Now that you have a QStringList, you can remove this "delete after the fact". Just move the new + setItems into the if (userNames.size() < maxEntries) - David Faure On March 26, 2014, 11:16 a.m., Alexander Richardson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/117078/ > ----------------------------------------------------------- > > (Updated March 26, 2014, 11:16 a.m.) > > > Review request for KDE Frameworks. > > > Repository: kio > > > Description > ------- > > A series of commits (can submit separately if necessary): > --- > Add S_IXUSR, IRUSR, etc. definitions for Windows > --- > Add KIOPrivate::isProcessAlive() and KIOPrivate::sendTerminateSignal() > > kill() does not work on Windows, therefore a WIN32 implementation is needed > in order to compile slave.cpp > --- > Add KIOPrivate::symlinkTarget() to avoid directly calling readlink() > > This allows KFileItem to be compiled on Windows. > The Windows implementation requires at least Windows Vista because > GetFinalPathNameByHandleW is not available in earlier version. Windows XP > support ends on the 8th of April so it should be okay to drop support for > it. If XP support is deemed necessary GetFinalPathNameByHandleW can be > loaded dynamically using GetProcAddress but that is not part of this patch. > Fix windows build of tests > --- > Add KIOPrivate::changeOwnership() to avoid directly calling chown() > > Additionally use KUserId/KGroupId instead of uid_t/gid_t. > This allows compiling chmodjob.cpp on Windows. > > KIOPrivate::changeOwnership is a stub on Windows. However, it has always > been like that with the kdewin chmod() implementation, so this is not a > regression from kdelibs4. > --- > Add KIOPrivate::createSymlink() to avoid using symlink() directly > --- > Minor Windows compile fixes > --- > Use KUser in KPropertiesDialog > > This means no more need for getpwent(), etc -> works on Windows > --- > Export the KIOPrivate functions > --- > Fix kio_http build on Windows > --- > No longer use uid_t/gid_t in kio_file > --- > Allow compiling kurlcompletion.cpp on windows > --- > Add a fake QT_LSTAT for Windows > > This just calls QT_STAT and adds QT_STAT_LNK flag to st_mode if it is a > symlink > --- > Use KUser in kpropertiesdialog.cpp -> no more getgrouplist > > > Diffs > ----- > > autotests/fileundomanagertest.cpp 3f209f89cc0e2ac48d8eaef7ee73ec18abca9a4c > autotests/globaltest.cpp 4367e53b44e077c566316081e21f429ac15b74a0 > autotests/kdirlistertest.cpp 10a970416b8874f1e136b460d05378f8f3a86810 > autotests/kfileitemtest.h dc1808e726cf6af1605cfda520c7df3832225cc1 > autotests/kfileitemtest.cpp 38bd87f4e9facd8e52e9e5fbd98b16011c866b5a > autotests/kiotesthelper.h eb9f0f3019deb63506c2a173d700b78daa95ae10 > src/core/CMakeLists.txt d897e370baedbe06b267934c123acee7a149adff > src/core/chmodjob.cpp 271869bc2a643d715670560b7920efdbc948e560 > src/core/job_error.cpp 1ec28c5d9ebdbc2045743c566f473c8cfa02e1bd > src/core/kfileitem.cpp 7364f87257b5d7dfb760b1c6e5b5d04e1d15a19d > src/core/kioglobal_p.h PRE-CREATION > src/core/kioglobal_p_unix.cpp PRE-CREATION > src/core/kioglobal_p_win.cpp PRE-CREATION > src/core/slave.cpp 787ffcf3cc97a73fb29c2172ed6b8df19ac016fc > src/ioslaves/file/file.h 6477df7cf0d26bf4f581151e1ce8e6c1115a221c > src/ioslaves/file/file.cpp 6387d7f29df2c0a26952be5255438361c37ed082 > src/ioslaves/file/file_win.cpp b0e433e5438e3c45f2f021bf073cb3cca8f4f923 > src/ioslaves/ftp/ftp.cpp 49037701772f88e9a96bc3110f1f04fccb810ca8 > src/ioslaves/http/http.cpp 3158628af2dc8ee88a0ac0d968748d6b50a8da33 > src/widgets/CMakeLists.txt 61e4db3566bad08baaa2e7e90b862ddfc8b957f7 > src/widgets/config-getgrouplist.h.cmake > 6847a19d60be4eb5c2b65fb86258f7368848e6ab > src/widgets/getgrouplist-fake.c dbe77067371dcedb80cea684fb3cd5f42ed72805 > src/widgets/kpropertiesdialog.cpp a3011a8adac98287bcada281dfc50b27e1af8fa0 > src/widgets/kurlcompletion.cpp 3f309257c187358de0fd66f9d67f09a712fdf7d6 > > Diff: https://git.reviewboard.kde.org/r/117078/diff/ > > > Testing > ------- > > compiles, tests still the same as before (i.e. not passing) > > > Thanks, > > Alexander Richardson > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel