Dear Gilles,<br>
<br>
Thanks for the feedback. I know that many checks must be performed in order to be sure that the application does not crash.<br>
<br>
I will look into the dmetadata and look the kipi plugin.<br>
<br>
Alex<br><br><div><span class="gmail_quote">On 9/18/06, <b class="gmail_sendername">Caulier Gilles</b> <<a href="mailto:caulier.gilles@kdemail.net">caulier.gilles@kdemail.net</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Le Dimanche 17 Septembre 2006 10:03 PM, Alexios Beveratos a écrit:<br>> Dear Gilles<br>><br>> Sure, you can look in my code, for the moment it is a bit messy and quite<br>> rudimentary. I will try to continue develop the following days
<br>><br><br>Just a speed look into your code : i have seen that you have'nt use the C++<br>exception to control each Exiv2 call. This is wrong because your app will<br>crash if something is wrong with exiv2.<br><br>Look the DMetadata class from digiKam core for example :
<br><br><a href="http://websvn.kde.org/trunk/extragear/graphics/digikam/libs/dmetadata/dmetadata.cpp?rev=576472&view=auto">http://websvn.kde.org/trunk/extragear/graphics/digikam/libs/dmetadata/dmetadata.cpp?rev=576472&view=auto
</a><br><br>Also, in kipi-plugins, i have started to implemente a common exiv2 interface<br>witch will be used by all plugins instead libkexif. I need to improve this<br>code, but in all case, if you want to create a kipi-plugin, i think that all
<br>your code witch use exiv2 library need to be moved in this interface.<br><br>Actually, the interface is only used by RAW converter. I will move this code<br>outside this plugin. The code is very simple actually, and will be improved
<br>accordinly with all kipi-plugins requirements.<br><br><a href="http://websvn.kde.org/trunk/extragear/libs/kipi-plugins/rawconverter/exiv2iface.cpp?rev=584763&view=auto">http://websvn.kde.org/trunk/extragear/libs/kipi-plugins/rawconverter/exiv2iface.cpp?rev=584763&view=auto
</a><br><br>Gilles<br><br>_______________________________________________<br>Digikam-devel mailing list<br><a href="mailto:Digikam-devel@kde.org">Digikam-devel@kde.org</a><br><a href="https://mail.kde.org/mailman/listinfo/digikam-devel">
https://mail.kde.org/mailman/listinfo/digikam-devel</a><br></blockquote></div><br>