DPL: fix a few warnings

Author: ktf

Framework/Core/include/Framework/GuiCallbackContext.h

@@ -23,7 +23,7 @@ namespace o2::framework
 {
 
 struct GuiCallbackContext;
-class WSDPLHandler;
+struct WSDPLHandler;
 
 struct GuiRenderer {
   uv_timer_t drawTimer;

Framework/Core/include/Framework/InitContext.h

@@ -16,7 +16,7 @@
 namespace o2::framework
 {
 
-class ServiceRegistry;
+struct ServiceRegistry;
 class ConfigParamRegistry;
 
 // This is a utility class to reduce the amount of boilerplate when defining

Framework/Core/include/Framework/ServiceRegistry.h

@@ -158,7 +158,7 @@ struct ServiceRegistry {
   /// not bonded to a specific stream, e.g. the
   /// name of the data processor, its inputs and outputs,
   /// it's algorithm.
-  static Salt dataProcessorSalt(short dataProcessorId)
+  static Salt dataProcessorSalt(short /* dataProcessorId */)
   {
     // FIXME: old behaviour for now
     // return {0, dataProcessorId};

Framework/Core/include/Framework/ServiceSpec.h

@@ -26,12 +26,12 @@ struct ProgOptions;
 namespace o2::framework
 {
 
-struct InitContext;
+class InitContext;
 struct DeviceSpec;
 struct ServiceRegistry;
-struct ServiceRegistryRef;
+class ServiceRegistryRef;
 struct DeviceState;
-struct ProcessingContext;
+class ProcessingContext;
 class EndOfStreamContext;
 struct ConfigContext;
 struct WorkflowSpecNode;

Framework/Core/include/Framework/StringHelpers.h

@@ -171,7 +171,7 @@ constexpr auto get_str(const char (&str)[N])
 }
 
 template <int N>
-constexpr auto get_size(const char (&str)[N])
+constexpr auto get_size(const char (&)[N])
 {
   return N;
 }

Framework/Core/src/TMessageSerializer.cxx

@@ -9,7 +9,7 @@
 // granted to it by virtue of its status as an Intergovernmental Organization
 // or submit itself to any jurisdiction.
 #include <Framework/TMessageSerializer.h>
-#include <FairMQTransportFactory.h>
+#include <fairmq/TransportFactory.h>
 #include <algorithm>
 #include <memory>