Re: Review Request 113045: Prepare KCMUtils for move to tier3

2013-10-04 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113045/#review41196 --- Ship it! Looks good, but why don't we replace KCategoryDrawer

Re: Review Request 113045: Prepare KCMUtils for move to tier3

2013-10-04 Thread David Edmundson
On Oct. 4, 2013, 7 a.m., David Faure wrote: Looks good, but why don't we replace KCategoryDrawer with KCategoryDrawerV3, in itemviews? I.e. getting rid of the two oldest ones? It should be SC since v3 derives from v2 derives from v1. It's not entirely source compatible since the

Re: Review Request 113045: Prepare KCMUtils for move to tier3

2013-10-04 Thread Commit Hook
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113045/#review41199 --- This review has been submitted with commit

Re: Review Request 113045: Prepare KCMUtils for move to tier3

2013-10-02 Thread Aleix Pol Gonzalez
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113045/#review41105 --- staging/kcmutils/CMakeLists.txt

Re: Review Request 113045: Prepare KCMUtils for move to tier3

2013-10-02 Thread David Edmundson
On Oct. 2, 2013, 11:02 a.m., Aleix Pol Gonzalez wrote: staging/kcmutils/CMakeLists.txt, line 23 http://git.reviewboard.kde.org/r/113045/diff/1/?file=193397#file193397line23 Why removing these? As I understand it everything should compile with -DQT_NO_CAST_TO_ASCII, it was

Re: Review Request 113045: Prepare KCMUtils for move to tier3

2013-10-02 Thread Stephen Kelly
David Edmundson wrote: On Oct. 2, 2013, 11:02 a.m., Aleix Pol Gonzalez wrote: staging/kcmutils/CMakeLists.txt, line 23 http://git.reviewboard.kde.org/r/113045/diff/1/?file=193397#file193397line23 Why removing these? As I understand it everything should compile with

Re: Review Request 113045: Prepare KCMUtils for move to tier3

2013-10-02 Thread David Edmundson
I haven't moved anything. Also this review is 3 different commits, one for each of the sentences in the description. ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Re: Review Request 113045: Prepare KCMUtils for move to tier3

2013-10-02 Thread Stephen Kelly
David Edmundson wrote: I haven't moved anything. Also this review is 3 different commits, one for each of the sentences in the description. Indeed, thanks! Steve. ___ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org