Review Request 113045: Prepare KCMUtils for move to tier3
Stephen Kelly
steveire at gmail.com
Wed Oct 2 12:16:41 UTC 2013
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 -DQT_NO_CAST_TO_ASCII,
> it was previously added here to make it compile before all the "blah" ->
> QLiteralString("blah") changes.
>
This discussion is exactly why you are supposed to make changes like this
before the move in a separate patch, so that you can explain the above in a
commit message.
Then the move commit is 'clean'.
Thanks,
Steve.
More information about the Kde-frameworks-devel
mailing list