Added MODE_NOLOG

[4c3y]

Added MODE_NOLOG to disable logging completely. Useful for unittests where logging can be ignored or in some cases for release builds.

• Implemented MODE_NOLOG
• Added Unittest suite for MODE_NOLOG
• Fixed and refactored some MODE_LPP unittests
• Fixed potential deadlock in async_tests.h

[michaelpantic]

lgtm!

It seems that in NoLog you log empty strings, right? Is there a technical difference in just redirecting the calls to not doing anything at all, to logging empty strings? With empty string you'd probably still transfer \0 to the stream, not sure if that makes a difference though.

[4c3y]

It actually doesn't log empty strings (The console would spam empty lines). The emptyString() function is constexpr and generates an empty string at compile-time, which is then passed to InternalLog() using a stream operator. This is useful to check at compile-time if the logging function signatures are valid, in case the code switched to a different mode (e.g. MODE_LPP). Inside InternalLog(), the following constructor and destructor are invoked:

class InternalLog() {
public:
  InternalLog() {
    should_print_ = false;
  };

 virtual ~InternalLog() {
    if (!should_print_) {
      return; //Returns here
    }
   //print logic here
  }
}

When utilizing the LPP_CAPTURE_STDOUT() macro during unittesting, an empty string is returned if nothing is printed.