----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112455/#review39275 -----------------------------------------------------------
Ship it! Once this is in, I think you can move netaccess to kde4support. kfile/knewfilemenu.cpp <http://git.reviewboard.kde.org/r/112455/#comment28966> if (!job->exec()) kfile/knewfilemenu.cpp <http://git.reviewboard.kde.org/r/112455/#comment28967> I've seen this before :-) Same thing about cutting out 6 lines. - David Faure On Sept. 3, 2013, 3 p.m., Àlex Fiestas wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112455/ > ----------------------------------------------------------- > > (Updated Sept. 3, 2013, 3 p.m.) > > > Review request for KDE Frameworks and David Faure. > > > Description > ------- > > Ported all remaining usages of KIO::NetAccess to Job+exec. > > In one case I have added a new method in the Private because the code looked > better. > > > Diffs > ----- > > kfile/kdiroperator.cpp 72789ae > kfile/kdirselectdialog.cpp 92cff2f > kfile/kfileplacesmodel.cpp e854059 > kfile/kfilewidget.cpp f668039 > kfile/knewfilemenu.cpp 5e768b8 > > Diff: http://git.reviewboard.kde.org/r/112455/diff/ > > > Testing > ------- > > > Thanks, > > Àlex Fiestas > >
_______________________________________________ Kde-frameworks-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
