Initialize function-local static variables using "T& t = *new T"

[lixingcong]

It would be safer if init the static object with reference according to Google C++ Style Guide.

I compile the static lib, link to my program, then get segmentation fault on calling mdc::get_context(). After applying these patches the bug was gone. :100:

[gabime]

Thanks but now tests fails with “LeakSanitizer: detected memory leaks”. A possible solution would be to delete those objects in spdlog::shutdown() and call it at the end of tests.

[lixingcong]

So hard to free memory of thread_local object created by T& t=*new T

#include <iostream>
#include <thread>

struct A {
    A(int i)
        : a(i)
    {
        std::cout << "ctor(), i=" << a << std::endl;
    }

    ~A()
    {
        std::cout << "dtor(), i=" << a << std::endl;
    }

    const int a;
};

static A& get_a()
{
    static thread_local A& a = *new A(8);
    static thread_local A  b(100);
    return a;
}

int main()
{
    std::thread t([]() { get_a(); });
    t.join();
    return 0;
}

Compile with gcc 11 it got weird print:

ctor(), i=8
ctor(), i=100
dtor(), i=100

That is why LeakSanitizer tell us memory leak happened.

[gabime]

This has to solved somehow