caulier.gilles at gmail.com
Tue Sep 30 09:10:05 BST 2014
2014-09-30 7:44 GMT+02:00 Gilles Caulier <caulier.gilles at gmail.com>:
> 2014-09-30 3:06 GMT+02:00 Vishesh Handa <me at vhanda.in>:
>> Hey Tobias
>> Some comments about the code -
>> 1. The code seems to be licensed under GPL. In order to make it into a
>> framework, it will need to be re-licensed. This library seems like an ideal
>> candidate for becoming a framework.
> libkface have been writted in same way than libkipi, libkexiv2, and
> libkdcraw, already in KDEGraphics.
>> 2. The copyright header seems to say "Part of the Digikam Project". You may
>> want to change that.
> Idem here. libkface follow exactly the same way than libkipi,
> libkexiv2, libkdcraw.
>> 3. There is an empty TODO file
> yes, this need to be filled.
>> 4. The coding style uses seems a little unorthdox. Could you perhaps add a
>> link to where one can know what style is being followed? Maybe this could go
>> in the README file.
> coding style follow instructions from digiKam project :
> It's the same coding style that libkexiv2, libkipi, and libkdcraw.
>> 5. Identity ABI - The Identity class seems to be missing a d pointer.
New bugzilla entry created with patch :
Tobias, this need to be tested. Also, PhotoAlbum code need to be adapted.
More information about the kde-core-devel