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. _______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel