Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Where does the destructor hide in this code?

I'm having trouble understanding why the Foo move constructor tries to invoke ~ptr in the following example:

#include <utility>

template <typename T, typename Policy>
class ptr {
    T * m_t;
public:
    ptr() noexcept : m_t(0) {}
    explicit ptr(T *t) noexcept : m_t(t) {}
    ptr(const ptr &other) noexcept : m_t(Policy::clone(other.m_t)) {}
    ptr(ptr &&other) noexcept : m_t(other.m_t) { other.m_t = 0; }
    ~ptr() noexcept { Policy::delete_(m_t); }
    ptr &operator=(const ptr &other) noexcept
    { ptr copy(other); swap(copy); return *this; }
    ptr &operator=(ptr &&other) noexcept
    { std::swap(m_t,other.m_t); return *this; }

    void swap(ptr &other) noexcept { std::swap(m_t, other.m_t); }

    const T * get() const noexcept { return m_t; }
    T * get() noexcept { return m_t; }
};

class FooPolicy;
class FooPrivate;
class Foo {
    // some form of pImpl:
    typedef ptr<FooPrivate,FooPolicy> DataPtr;
    DataPtr d;
public:
    // copy semantics: out-of-line
    Foo();
    Foo(const Foo &other);
    Foo &operator=(const Foo &other);
    ~Foo();

    // move semantics: inlined
    Foo(Foo &&other) noexcept
      : d(std::move(other.d)) {} // l.35 ERR: using FooDeleter in ~ptr required from here
    Foo &operator=(Foo &&other) noexcept
    { d.swap(other.d); return *this; }
};

GCC 4.7:

foo.h: In instantiation of ‘ptr<T, Policy>::~ptr() [with T = FooPrivate; Policy = FooPolicy]’:
foo.h:34:44:   required from here
foo.h:11:14: error: incomplete type ‘FooPolicy’ used in nested name specifier

Clang 3.1-pre:

foo.h:11:14: error: incomplete type 'FooPolicy' named in nested name specifier
    ~ptr() { Policy::delete_(m_t); }
             ^~~~~~~~
foo.h:34:5: note: in instantiation of member function 'ptr<FooPrivate, FooPolicy>::~ptr' requested here
    Foo(Foo &&other) : d(std::move(other.d)) {}
    ^
foo.h:23:7: note: forward declaration of 'FooPolicy'
class FooPolicy;
      ^
foo.h:11:20: error: incomplete definition of type 'FooPolicy'
    ~ptr() { Policy::delete_(m_t); }
             ~~~~~~^~
2 errors generated.

What's going on? I'm writing move constructors to avoid running copy ctors and dtors. Note that this is a header file that tries to hide its implementation (pimpl idiom), so making FooDeleter a full type isn't an option.

EDIT: After Bo's answer, I added noexcept everywhere I could (edited in above). But the errors are still the same.

like image 862
Marc Mutz - mmutz Avatar asked Feb 23 '12 16:02

Marc Mutz - mmutz


Video Answer


1 Answers

You create a new Foo object that contains a ptr<something> member. In case the Foo constructor fails, the compiler has to call the destructors for any completely constructed members of the partially constructed Foo.

But it cannot instantiate ~ptr<incomplete_type>(), so that fails.

You have a similar case with a private destructor. That also stops you from creating objects of that type (unless done from a friend or a member function).

like image 133
Bo Persson Avatar answered Oct 06 '22 14:10

Bo Persson