Moving KScreen and libkscreen to extragear

[David Edmundson]

Code wise, things looks pretty good.

Minor comments:
 - the library is GPL, not LGPL which is the norm for libraries.
Is this deliberate?

 - Inside kscreen you have a copy of the metadata.desktop file twice.
Just have the one and install it to the two places.
You might also want to use the version number from your main CMakeLists.txt
in your .desktop file too, otherwise updating can get messy.

-Generator::biggestOutput
rename the local variable "total"...it's not a total of anything, I had to
re-read it 10 times to realise the code was correct.


In general though +1 from me.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130708/b00dec77/attachment.htm>