[Digikam-devel] specific KDE Version test in editorwindow.cpp

[Andi Clemens]

I added this to have grouping of actions in the shortcut editor. You are 
right, the include must be enclosed as well, I forgot that.
Will add this now!

Andi Clemens
-----------------
www.digikam.org

On Saturday 12 December 2009 18:25:28 Arnd Baecker wrote:
> Hi,
> 
> in void EditorWindow::loadImagePlugins()
> there is a very specific:
>   #if KDE_IS_VERSION(4,1,68)
> which makes use of
>   QString categoryStr = plugin->actionCategory();
> 
> Does anyone know what the reason for this special case is
> (and whether it is really necessary?)?
> 
> And if yes, shouldn't then the include at the beginning be
> encapsulated as well, i.e.:
> #if KDE_IS_VERSION(4,1,68)
>    #include <kactioncategory.h>
> #endif
> ?
> (Background info: I neither have KDE 4.1.68, nor kactioncategory.h,
> so the compile failed for me; commenting `#include <kactioncategory.h>`
> out got me a bit further ...;-) )
> 
> Best, Arnd
> _______________________________________________
> Digikam-devel mailing list
> Digikam-devel at kde.org
> https://mail.kde.org/mailman/listinfo/digikam-devel
>