----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/120460/#review68746 -----------------------------------------------------------
Ship it! Ship It! - David Faure On Oct. 20, 2014, 8:35 a.m., Emmanuel Pescosta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/120460/ > ----------------------------------------------------------- > > (Updated Oct. 20, 2014, 8:35 a.m.) > > > Review request for kde-workspace and David Faure. > > > Repository: kio-extras > > > Description > ------- > > The filenamesearch ioslave lists all files/folders which match with the given > search query. > > Made some small adjustments: > * Ported the ioslave to Qt5/KF5 > * Got rid of cleanup() and member variables > * Use lambda function validators instead of some weird if + assert things in > searchDirectory/contentContainsPattern > * Immediately return when the search string is empty > > https://projects.kde.org/projects/kde/applications/kde-baseapps/repository/revisions/frameworks/show/dolphin/src/search > > > Diffs > ----- > > CMakeLists.txt 5ceeaf1 > filenamesearch/CMakeLists.txt PRE-CREATION > filenamesearch/filenamesearch.protocol PRE-CREATION > filenamesearch/kio_filenamesearch.h PRE-CREATION > filenamesearch/kio_filenamesearch.cpp PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/120460/diff/ > > > Testing > ------- > > Compiles. > > > Thanks, > > Emmanuel Pescosta > >
