> On June 14, 2013, 7:02 a.m., Kevin Ottens wrote: > > OK, this one is way too big. Could you please upload that as two separate > > reviews? One containing the changes to the class (all the porting work you > > did), and one containing just the file moves with the corresponding > > CMakeLists.txt adjustment. > > > > Otherwise it'll be hard to properly review and we're bound to miss > > something.
You can generate the diff using: git diff --find-copies-harder The patch will contain the file moves themselves - Aleix ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/110995/#review34335 ----------------------------------------------------------- On June 13, 2013, 10:29 p.m., Wojciech Kapuscinski wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/110995/ > ----------------------------------------------------------- > > (Updated June 13, 2013, 10:29 p.m.) > > > Review request for KDE Frameworks, David Faure and Kevin Ottens. > > > Description > ------- > > move jobs from kdeui to kjobswidgets framework > > Port away from: > NET::timestampCompare > KWindowSystem::setIcon (QWidget.setWindowIcon) > ki18n (not sure if KWidgetJobTracker::Private::ProgressWidget::description() > is correctly ported) > KStandardGuiItem > > I'm not sure about plural translations (appears (s) because there is not > English translation) > > > Diffs > ----- > > kdeui/CMakeLists.txt 46196b4 > kdeui/jobs/kabstractwidgetjobtracker.h 892e787 > kdeui/jobs/kabstractwidgetjobtracker.cpp 0b9e829 > kdeui/jobs/kabstractwidgetjobtracker_p.h 5f427e3 > kdeui/jobs/kdialogjobuidelegate.h 186f160 > kdeui/jobs/kdialogjobuidelegate.cpp 5afe6ec > kdeui/jobs/kjobtrackerformatters.cpp 95e8cfb > kdeui/jobs/kjobtrackerformatters_p.h aeeb856 > kdeui/jobs/kstatusbarjobtracker.h dbab452 > kdeui/jobs/kstatusbarjobtracker.cpp 688c942 > kdeui/jobs/kstatusbarjobtracker_p.h 9d93e9c > kdeui/jobs/kwidgetjobtracker.h 761267f > kdeui/jobs/kwidgetjobtracker.cpp 835cad0 > kdeui/jobs/kwidgetjobtracker_p.h 2a3f297 > kdeui/tests/CMakeLists.txt 8746397 > kdeui/tests/kjobtrackerstest.h c6f3195 > kdeui/tests/kjobtrackerstest.cpp bbde166 > staging/kjobwidgets/src/CMakeLists.txt 84756ce > staging/kjobwidgets/src/config-kjobwidgets.h.cmake PRE-CREATION > staging/kjobwidgets/src/kabstractwidgetjobtracker.h PRE-CREATION > staging/kjobwidgets/src/kabstractwidgetjobtracker.cpp PRE-CREATION > staging/kjobwidgets/src/kabstractwidgetjobtracker_p.h PRE-CREATION > staging/kjobwidgets/src/kdialogjobuidelegate.h PRE-CREATION > staging/kjobwidgets/src/kdialogjobuidelegate.cpp PRE-CREATION > staging/kjobwidgets/src/kjobtrackerformatters.cpp PRE-CREATION > staging/kjobwidgets/src/kjobtrackerformatters_p.h PRE-CREATION > staging/kjobwidgets/src/kstatusbarjobtracker.h PRE-CREATION > staging/kjobwidgets/src/kstatusbarjobtracker.cpp PRE-CREATION > staging/kjobwidgets/src/kstatusbarjobtracker_p.h PRE-CREATION > staging/kjobwidgets/src/kuiserverjobtracker.cpp 13de2f4 > staging/kjobwidgets/src/kwidgetjobtracker.h PRE-CREATION > staging/kjobwidgets/src/kwidgetjobtracker.cpp PRE-CREATION > staging/kjobwidgets/src/kwidgetjobtracker_p.h PRE-CREATION > staging/kjobwidgets/tests/CMakeLists.txt d41be07 > staging/kjobwidgets/tests/kjobtrackerstest.h PRE-CREATION > staging/kjobwidgets/tests/kjobtrackerstest.cpp PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/110995/diff/ > > > Testing > ------- > > it builds and my tests shows now regressions (except plural translations (s) ) > > > Thanks, > > Wojciech Kapuscinski > >
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
