[Development] Reusing Qt code (or: saving .ico files)

[René J.V. Bertin]

Hello,

Can someone please have a look at the KDE RR below and tell us whether or not it's OK to include qicohandler.* the way I propose in the patch under review? There are some whitespace changes and I added a preprocessor token to exclude ico read support.
My understanding of the GPL legalese is that it should be OK but Christoph is right that we have to double-check.

Maybe we all overlooked something and there is another way to create .ico files, without needing to use (and thus include) QtIcoHandler explicitly? I see that the relevant QtBase code is built on Linux and Mac but I cannot tell whether it is also installed (the header isn't in any case).

Thanks,
René

-------------------------------
Forwarded message:
Date: Friday January 13 2017
From: Christoph Feck <cfeck at kde.org>
Subject: Re: Review Request 129807: proposal: add ksvg2ico

https://git.reviewboard.kde.org/r/129807/#review101968


Someone more experienced with legal matters should review the verbatim inclusion of Qt code; in particular the license header.

- Christoph Feck