Отладочная замена для make_shared()

Используя gcc 4.6.2, make_shared() дает бесполезную обратную трассировку (видимо, из-за некоторого повтора), если конструктор генерирует исключение. Я использую make_shared(), чтобы сохранить немного ввода, но это шоу-стоп. Я создал замену make_shrd(), которая позволяет нормальную обратную трассировку. Я использую gdb 7.3.1.

Я беспокоюсь, что:

  • Плохая обратная трассировка под make_shared() - это как-то моя собственная ошибка.
  • Мой заместитель make_shrd() вызовет у меня тонкие проблемы.

Вот демо:

#include <memory>
#include <stdexcept>

using namespace std;

class foo1
{
public:
        foo1( const string& bar, int x ) :m_bar(bar), m_x(x)
        {
                throw logic_error( "Huh?" );
        }
        string m_bar;
        int m_x;
};

class foo2
{
public:
        foo2( const string& bar, int x ) : m_foo1(bar,x)
        {}

        foo1  m_foo1;
};

// more debuggable substitute for make_shared() ??
template<typename T, typename... Args>
std::shared_ptr<T> make_shrd( Args... args )
{
        return std::shared_ptr<T>( new T(args...));
}

int main()
{
        auto p_foo2 = make_shared<foo2>( "stuff", 5 );          // debug BAD!!
//      auto p_foo2 = make_shrd<foo2>( "stuff", 5 );            // debug OK
//      auto p_foo2 = new foo2( "stuff", 5 );                   // debug OK
//      auto p_foo2 = shared_ptr<foo2>(new foo2( "stuff", 5 )); // debug OK
        return (int)(long int)p_foo2;
}

Скомпилировано с помощью

g++ -g -std=c++0x -Wall -Wextra main.cpp

Отладка с помощью:

gdb a.out

Make_shared() backtrace - это мусор, который не показывает стек до точки исключения. Все остальные параметры обеспечивают надежную обратную трассировку.

Заранее благодарим за помощь и предложения.

Ответ 1

Ваша реализация make_shrd() теряет способность выделять только один кусок памяти: std::make_shared() выполняет две вещи:

  • он избегает дублирования написания типа (если тип выделения и тип желаемого std::shared_ptr<T> совпадают, а не последний для базового класса)
  • он объединяет выделение общего объекта и дескриптора объекта только в одно размещение

Основная цель std::make_shared() - фактически вторая функция. Я не смотрел на реализацию, но я подозреваю, что это тоже та часть, которая на самом деле вызывает проблемы. Кроме этого, я не вижу причин, по которым ваша реализация хуже, когда вы исправляете пересылку аргументов:

template<typename T, typename... Args>
std::shared_ptr<T> make_shrd(Args&&... args)
{
    return std::shared_ptr<T>(new T(std::forward<Args>(args)...));
}