Bug 339524 - Identity class miss d private container to improve binary compatibility [patch]
Summary: Identity class miss d private container to improve binary compatibility [patch]
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Faces-Detection (show other bugs)
Version: 4.4.0
Platform: Other All
: NOR wishlist
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-09-30 08:01 UTC by caulier.gilles
Modified: 2014-10-11 13:52 UTC (History)
4 users (show)

See Also:
Latest Commit: http://commits.kde.org/libkface/cc6a4ca7d7b61eeebe8438bcec2a93659af2a745
Version Fixed In: 4.5.0

Attachments
patch for libkface (10.95 KB, patch)
2014-09-30 08:08 UTC, caulier.gilles 		Details
patch for digiKam (3.35 KB, patch)
2014-09-30 08:21 UTC, caulier.gilles 		Details
patch for libkface (v2) (11.01 KB, patch)
2014-09-30 12:25 UTC, caulier.gilles 		Details
Show Obsolete (1) View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description caulier.gilles 2014-09-30 08:01:06 UTC 
In order to preserve binary compatibility, Identity class need new accessor methods to play with internal members. Currently all members are public. This need to be changed as other class from libkface.

Reproducible: Always
Comment 1 caulier.gilles 2014-09-30 08:08:11 UTC 
Created attachment 88899 [details]
patch for libkface

Path to make internal members private and add new methods to access on private members.
Comment 2 caulier.gilles 2014-09-30 08:13:15 UTC 
Marcel, 

There is a current discussion into kde-core-devel@kde.org mailing list to make libkface more public into kdegraphics component, as libkipi, libkexiv2, and libkdcraw.

This patch increase BC into Identity class in order to make this library public.

Can you check my patches please to see if all is fine for you.

Thanks in advance

Gilles Caulier
Comment 3 caulier.gilles 2014-09-30 08:21:04 UTC 
Created attachment 88900 [details]
patch for digiKam

digiKam patch to use new libkface API
Comment 4 Marcel Wiesweg 2014-09-30 12:11:23 UTC 
Comment on attachment 88899 [details]
patch for libkface

If we really believe we need this redirection, then the patch is ok I think
Comment 5 caulier.gilles 2014-09-30 12:25:39 UTC 
Created attachment 88902 [details]
patch for libkface (v2)

Some code polish with map iterator used in Identity attibutes loop
Comment 6 Tobias Leupold 2014-09-30 16:26:33 UTC 
Gilles, With v2 of your patch, KPA does not crash anymore. Everything works as before (with the patches necessary to respect the new API of course). Thanks :-)
Comment 7 caulier.gilles 2014-09-30 16:29:55 UTC 
Thanks Tobias,

I propose to wait digiKam 4.4.0 release planed next Sunday evening before to apply this patch on git/master.

Gilles
Comment 8 Tobias Leupold 2014-09-30 16:43:18 UTC 
I'm fine with that. I'll patch KPA git master as soon as your changes are in libkface git, and I know which version cmake has to check (at the moment, it's >=3.3.0). Perhaps you could drop me an email?

Nice to see that libkface is on it's way to be an "independent" library with an own release :-)
Comment 9 caulier.gilles 2014-10-11 09:19:06 UTC 
Patches applied to git/master.

Gilles Caulier