[Digikam-devel] [Fwd: Re: [Bug 187291] FTP KIPI plugin for uploading a selection of images]

[Johannes Wienke]

For some strange reason my mails don't get on the list, so here once 
again without gpg signature. Maybe that's the problem...

Johannes

-------- Original-Nachricht --------
Betreff: Re: [Digikam-devel] [Bug 187291] FTP KIPI plugin for uploading 
a selection of images
Datum: Tue, 29 Sep 2009 18:02:05 +0200
Von: Johannes Wienke <languitar at semipol.de>
An: digiKam developers room - digiKam: Manage your photographs as a 
professional with the power of open source <digikam-devel at kde.org>
Referenzen: <bug-187291-42497 at http.bugs.kde.org/> 
<20090929153237.C74AF2192C at immanuel.kde.org>

Hi,

regarding this commit, I have some questions concerning the coding style
I found in the other plugins that I adopted for this plugins:

1. What's the reason for using forward declarations of other KIPI
classes instead of includes in the header files?

2. Is there any preferred way for accessing KConfig? In one plugin I
found a solution using "kipirc" as name for the constructor, another
plugin simply used KGlobal::config().

3. Is it possible that KConfigGroups cannot be reassigned? I tried using
a function like this:
KConfigGroup getConfigGroup() {
      return KGlobal::config()->group("a_name");
}
An when I used it in another function I got segfaults somewhere deep
inside KConfig.

Johannes

Am 29.09.2009 17:32 schrieb Andi Clemens:
> https://bugs.kde.org/show_bug.cgi?id=187291
> 
> 
> Andi Clemens <andi.clemens at gmx.net> changed:
> 
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>              Status|UNCONFIRMED                 |RESOLVED
>          Resolution|                            |FIXED
> 
> 
> 
> 
> --- Comment #6 from Andi Clemens <andi clemens gmx net>  2009-09-29 17:32:34 ---
> SVN commit 1029357 by aclemens:
> 
> Add new KIPIplugin "KioExport".
> This plugin can be used to export images to FTP, Samba, or just local paths
> using the KIO technologies.
> 
> Thanks to Johannes Wienke for the patch.
> 
> BUG:187291
> 
>  M  +1 -0      CMakeLists.txt  
>  A             kioexport (directory)  
>  A             kioexport/CMakeLists.txt  
>  A             kioexport/KioExportWidget.cpp   [License: GPL (v2+)]
>  A             kioexport/KioExportWidget.h   [License: GPL (v2+)]
>  A             kioexport/KioExportWindow.cpp   [License: GPL (v2+)]
>  A             kioexport/KioExportWindow.h   [License: GPL (v2+)]
>  A             kioexport/Messages.sh  
>  A             kioexport/kipiplugin_kioexport.desktop  
>  A             kioexport/plugin_kioexport.cpp   [License: GPL (v2+)]
>  A             kioexport/plugin_kioexport.h   [License: GPL (v2+)]
> 
> 
> WebSVN link: http://websvn.kde.org/?view=rev&revision=1029357
>