Code wise, things looks pretty good.<div><br></div><div>Minor comments:</div><div> - the library is GPL, not LGPL which is the norm for libraries.</div><div>Is this deliberate?</div><div><br></div><div> - Inside kscreen you have a copy of the metadata.desktop file twice.</div>
<div>Just have the one and install it to the two places. </div><div>You might also want to use the version number from your main CMakeLists.txt in your .desktop file too, otherwise updating can get messy.</div><div><br></div>
<div>-Generator::biggestOutput</div><div>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.</div><div><br></div><div><br></div><div>
In general though +1 from me.</div>