Is the logging lib supposed to be stack based?

[aadeg]

Looking at the source code of the logging module, I discover that it uses the heap. For example the LogEntry struct is defined as:

struct LogEntry
{
    LogLevel level{LogLevel::kVerbose};
    std::chrono::milliseconds time{0};
    std::string message;   //  <------- heap
};

Isn't it against the rules written in the CONTRIBUTION.md?

1. No heap is allowed, static memory management hugely decreases the complexity of your software (e.g. cxx::vector without heap)

[elBoberido]

@aadeg this was just a stop gap solution. I'm currently working on a design document to overhaul the logger and also remove the heap usage. There are a few other places where currently a std::string is used. All of this will be fixed in the future.

[aadeg]

Great news! Thanks for the clarification