Review Request 110995: move jobs from kdeui to kjobswidgets framework

Aleix Pol Gonzalez aleixpol at kde.org
Fri Jun 14 10:28:42 UTC 2013



> 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
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130614/b8059f6a/attachment.html>


More information about the Kde-frameworks-devel mailing list