diff options
Diffstat (limited to 'src/utility')
-rw-r--r-- | src/utility/logger.cc | 21 | ||||
-rw-r--r-- | src/utility/logger.h | 33 |
2 files changed, 30 insertions, 24 deletions
diff --git a/src/utility/logger.cc b/src/utility/logger.cc new file mode 100644 index 0000000..cb5c79a --- /dev/null +++ b/src/utility/logger.cc @@ -0,0 +1,21 @@ +#include "logger.h" + +namespace utility { + +// Forward the contents of a given standard output stream to the log target +// +// While `this->stream_ << stream.rdbuf()` would be more effective it sadly +// does not work with `boost::process::pistream` due to a broken pipe error +// in conjunction with the required `boost::process::capture_stream` context +// flag. +// +void Logger::forward(boost::process::pistream& stream) { + std::lock_guard<std::mutex> guard(this->write_mutex_); + + this->stream_ << std::string( + (std::istreambuf_iterator<char>(stream)), + (std::istreambuf_iterator<char>()) + ); +} + +} diff --git a/src/utility/logger.h b/src/utility/logger.h index 6fd6613..49b7cc1 100644 --- a/src/utility/logger.h +++ b/src/utility/logger.h @@ -10,45 +10,30 @@ namespace utility { class Logger { - #define FOR_EACH_ARGUMENT(...)\ - auto&& tmp = { (__VA_ARGS__, 0)... }; (void) tmp; - public: Logger(const int target_fd): buffer_(target_fd, std::ios::out), stream_(&this->buffer_) { } - template <typename... Args> - void append(const Args&... args) { - std::lock_guard<std::mutex> guard(this->write_mutex_); + template <typename Head> + inline void append(Head&& head) { + this->stream_ << head << std::endl; + } - FOR_EACH_ARGUMENT(this->stream_ << args); + template <typename Head, typename... Tail> + inline void append(Head&& head, Tail&&... tail) { + this->stream_ << head; - this->stream_ << std::endl; + this->append(std::forward<Tail>(tail)...); } - // Forward the contents of a given standard output stream to the log target - // - // While `this->stream_ << stream.rdbuf()` would be more effective it sadly - // does not work with `boost::process::pistream` due to a broken pipe error - // in conjunction with the required `boost::process::capture_stream` context - // flag. - // - void forward(boost::process::pistream& stream) { - std::lock_guard<std::mutex> guard(this->write_mutex_); - - this->stream_ << std::string( - (std::istreambuf_iterator<char>(stream)), - (std::istreambuf_iterator<char>()) - ); - } + void forward(boost::process::pistream&); private: __gnu_cxx::stdio_filebuf<char> buffer_; std::ostream stream_; std::mutex write_mutex_; - #undef FOR_EACH_ARGUMENT }; } |