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

Arnd Baecker arnd.baecker at web.de
Sat Dec 12 17:25:28 GMT 2009


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



More information about the Digikam-devel mailing list