Convert to a global static logger

Before this commit, logger pointers get passed around through
inheritance and manually constructed dependency assignment lists.
The manual management is hard to scale with logging calls which
can appear anywhere in the code.

This commit implements a single global static logger for all
Freenect2 contexts. It still can be replaced by different
loggers, but only one at a time.

Now it is the responsibility of each logging point to include
libfreenect2/logging.h, which is not automatically included.

Author: xlz

examples/protonect/Protonect.cpp

@@ -33,6 +33,7 @@
 #include <libfreenect2/threading.h>
 #include <libfreenect2/registration.h>
 #include <libfreenect2/packet_pipeline.h>
+#include <libfreenect2/logging.h>
 #ifdef LIBFREENECT2_WITH_OPENGL_SUPPORT
 #include "viewer.h"
 #endif
@@ -59,7 +60,7 @@ int main(int argc, char *argv[])
 
   libfreenect2::Freenect2 freenect2;
   // create a console logger with debug level (default is console logger with info level)
-  freenect2.setLogger(libfreenect2::createConsoleLogger(libfreenect2::Logger::Debug));
+  libfreenect2::setGlobalLogger(libfreenect2::createConsoleLogger(libfreenect2::Logger::Debug));
 
   libfreenect2::Freenect2Device *dev = 0;
   libfreenect2::PacketPipeline *pipeline = 0;

examples/protonect/include/libfreenect2/depth_packet_stream_parser.h

@@ -31,7 +31,6 @@
 #include <stdint.h>
 
 #include <libfreenect2/config.h>
-#include <libfreenect2/logging.h>
 
 #include <libfreenect2/double_buffer.h>
 #include <libfreenect2/depth_packet_processor.h>
@@ -52,7 +51,7 @@ LIBFREENECT2_PACK(struct LIBFREENECT2_API DepthSubPacketFooter
   uint32_t fields[32];
 });
 
-class LIBFREENECT2_API DepthPacketStreamParser : public DataCallback, public WithLoggerImpl
+class LIBFREENECT2_API DepthPacketStreamParser : public DataCallback
 {
 public:
   DepthPacketStreamParser();

examples/protonect/include/libfreenect2/libfreenect2.hpp

@@ -29,7 +29,6 @@
 
 #include <libfreenect2/config.h>
 #include <libfreenect2/frame_listener.hpp>
-#include <libfreenect2/logging.h>
 
 namespace libfreenect2
 {
@@ -96,15 +95,12 @@ class LIBFREENECT2_API Freenect2Device
 
 class Freenect2Impl;
 
-class LIBFREENECT2_API Freenect2 : public WithLogger
+class LIBFREENECT2_API Freenect2
 {
 public:
   Freenect2(void *usb_context = 0);
   virtual ~Freenect2();
 
-  virtual void setLogger(Logger *logger);
-  virtual Logger *logger();
-
   int enumerateDevices();
 
   std::string getDeviceSerialNumber(int idx);

examples/protonect/include/libfreenect2/logging.h

@@ -60,6 +60,10 @@ LIBFREENECT2_API Logger *createConsoleLogger(Logger::Level level);
 LIBFREENECT2_API Logger *createConsoleLoggerWithDefaultLevel();
 LIBFREENECT2_API Logger *createNoopLogger();
 
+//libfreenect2 frees the memory of the logger passed in.
+LIBFREENECT2_API Logger *getGlobalLogger();
+LIBFREENECT2_API void setGlobalLogger(Logger *logger);
+
 class LIBFREENECT2_API LogMessage
 {
 private:
@@ -73,50 +77,22 @@ class LIBFREENECT2_API LogMessage
   std::ostream &stream();
 };
 
-class LIBFREENECT2_API WithLogger
-{
-public:
-  virtual ~WithLogger();
-  virtual void setLogger(Logger *logger) = 0;
-  virtual Logger *logger() = 0;
-};
-
-template<class T>
-void trySetLogger(T *obj, Logger *logger)
-{
-  WithLogger *obj_with_logger = dynamic_cast<WithLogger *>(obj);
-  if(obj_with_logger != 0)
-  {
-    obj_with_logger->setLogger(logger);
-  }
-}
-
-class LIBFREENECT2_API WithLoggerImpl : public WithLogger
-{
-protected:
-  Logger *logger_;
-
-  virtual void onLoggerChanged(Logger *logger);
-public:
-  WithLoggerImpl();
-  virtual ~WithLoggerImpl();
-  virtual void setLogger(Logger *logger);
-  virtual Logger *logger();
-};
+std::string getShortName(const char *func);
 
 } /* namespace libfreenect2 */
 
 #if defined(__GNUC__) or defined(__clang__)
-#define LIBFREENECT2_LOG_SOURCE __PRETTY_FUNCTION__
+#define LIBFREENECT2_LOG_SOURCE ::libfreenect2::getShortName(__PRETTY_FUNCTION__)
 #elif defined(_MSC_VER)
-#define LIBFREENECT2_LOG_SOURCE __FUNCSIG__
+#define LIBFREENECT2_LOG_SOURCE ::libfreenect2::getShortName(__FUNCSIG__)
 #else
 #define LIBFREENECT2_LOG_SOURCE ""
 #endif
 
-#define LOG_DEBUG (::libfreenect2::LogMessage(logger(), ::libfreenect2::Logger::Debug).stream() << "[" << LIBFREENECT2_LOG_SOURCE << "] ")
-#define LOG_INFO (::libfreenect2::LogMessage(logger(), ::libfreenect2::Logger::Info).stream() << "[" << LIBFREENECT2_LOG_SOURCE << "] ")
-#define LOG_WARNING (::libfreenect2::LogMessage(logger(), ::libfreenect2::Logger::Warning).stream() << "[" << LIBFREENECT2_LOG_SOURCE << "] ")
-#define LOG_ERROR (::libfreenect2::LogMessage(logger(), ::libfreenect2::Logger::Error).stream() << "[" << LIBFREENECT2_LOG_SOURCE << "] ")
+#define LOG(LEVEL) (::libfreenect2::LogMessage(::libfreenect2::getGlobalLogger(), ::libfreenect2::Logger::LEVEL).stream() << "[" << LIBFREENECT2_LOG_SOURCE << "] ")
+#define LOG_DEBUG LOG(Debug)
+#define LOG_INFO LOG(Info)
+#define LOG_WARNING LOG(Warning)
+#define LOG_ERROR LOG(Error)
 
 #endif /* LOGGING_H_ */

examples/protonect/include/libfreenect2/packet_pipeline.h

@@ -28,7 +28,6 @@
 #define PACKET_PIPELINE_H_
 
 #include <libfreenect2/config.h>
-#include <libfreenect2/logging.h>
 #include <libfreenect2/data_callback.h>
 #include <libfreenect2/rgb_packet_stream_parser.h>
 #include <libfreenect2/depth_packet_stream_parser.h>
@@ -51,7 +50,7 @@ class LIBFREENECT2_API PacketPipeline
   virtual DepthPacketProcessor *getDepthPacketProcessor() const = 0;
 };
 
-class LIBFREENECT2_API BasePacketPipeline : public PacketPipeline, public WithLoggerImpl
+class LIBFREENECT2_API BasePacketPipeline : public PacketPipeline
 {
 protected:
   RgbPacketStreamParser *rgb_parser_;
@@ -64,8 +63,6 @@ class LIBFREENECT2_API BasePacketPipeline : public PacketPipeline, public WithLo
 
   virtual void initialize();
   virtual DepthPacketProcessor *createDepthPacketProcessor() = 0;
-
-  virtual void onLoggerChanged(Logger *logger);
 public:
   virtual ~BasePacketPipeline();
 

examples/protonect/include/libfreenect2/protocol/command_transaction.h

@@ -28,15 +28,14 @@
 #define COMMAND_TRANSACTION_H_
 
 #include <libusb.h>
-#include <libfreenect2/logging.h>
 #include <libfreenect2/protocol/command.h>
 
 namespace libfreenect2
 {
 namespace protocol
 {
 
-class CommandTransaction : public WithLoggerImpl
+class CommandTransaction
 {
 public:
   static const int ResponseCompleteLength = 16;
@@ -64,7 +63,7 @@ class CommandTransaction : public WithLoggerImpl
   };
 
   CommandTransaction(libusb_device_handle *handle, int inbound_endpoint, int outbound_endpoint);
-  virtual ~CommandTransaction();
+  ~CommandTransaction();
 
   void execute(const CommandBase& command, Result& result);
 private:

examples/protonect/include/libfreenect2/protocol/usb_control.h

@@ -28,7 +28,6 @@
 #define USB_CONTROL_H_
 
 #include <libusb.h>
-#include <libfreenect2/logging.h>
 
 namespace libfreenect2
 {
@@ -62,7 +61,7 @@ namespace protocol
  *
  * The 2. interface can be enabled/disabled by changing its alternate setting to 1/0
  */
-class UsbControl : public WithLoggerImpl
+class UsbControl
 {
 public:
   UsbControl(libusb_device_handle *handle);

examples/protonect/include/libfreenect2/rgb_packet_stream_parser.h

@@ -30,7 +30,6 @@
 #include <stddef.h>
 
 #include <libfreenect2/config.h>
-#include <libfreenect2/logging.h>
 #include <libfreenect2/double_buffer.h>
 #include <libfreenect2/rgb_packet_processor.h>
 
@@ -39,7 +38,7 @@
 namespace libfreenect2
 {
 
-class LIBFREENECT2_API RgbPacketStreamParser : public DataCallback, public WithLoggerImpl
+class LIBFREENECT2_API RgbPacketStreamParser : public DataCallback
 {
 public:
   RgbPacketStreamParser();

examples/protonect/include/libfreenect2/usb/transfer_pool.h

@@ -30,7 +30,6 @@
 #include <vector>
 #include <libusb.h>
 
-#include <libfreenect2/logging.h>
 #include <libfreenect2/data_callback.h>
 #include <libfreenect2/threading.h>
 
@@ -40,7 +39,7 @@ namespace libfreenect2
 namespace usb
 {
 
-class TransferPool : public WithLoggerImpl
+class TransferPool
 {
 public:
   TransferPool(libusb_device_handle *device_handle, unsigned char device_endpoint);

examples/protonect/src/command_transaction.cpp

@@ -25,6 +25,7 @@
  */
 
 #include <libfreenect2/protocol/command_transaction.h>
+#include <libfreenect2/logging.h>
 
 #include <stdint.h>