chinmoyr updated this revision to Diff 17202. chinmoyr marked 5 inline comments as done. chinmoyr added a comment.
1. Updated doc 2. Renamed signal to privilegeOperationRequested(). 3. Merged request(signal) and reply into one method, isPrivilegeOperationAllowed(). Now there is no round trip. 4. Eliminated QStringList. CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D6829?vs=17017&id=17202 BRANCH temp REVISION DETAIL https://phabricator.kde.org/D6829 AFFECTED FILES src/core/slavebase.cpp src/core/slavebase.h src/core/slaveinterface.cpp src/core/slaveinterface.h src/ioslaves/file/CMakeLists.txt src/ioslaves/file/file.h src/ioslaves/file/file_unix.cpp src/ioslaves/file/file_win.cpp src/ioslaves/file/kauth/CMakeLists.txt src/ioslaves/file/kauth/file.actions src/ioslaves/file/kauth/filehelper.cpp src/ioslaves/file/kauth/filehelper.h src/ioslaves/file/sharefd.cpp src/ioslaves/file/sharefd.h To: chinmoyr, dfaure, #frameworks Cc: #frameworks