Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

C++ Arrays and make_unique

As a follow up to this post I wonder how its implementation of make_unique plays with allocating function-temporary buffer arrays such as in the following code.

f()
{
  auto buf = new int[n]; // temporary buffer
  // use buf ...
  delete [] buf;
}

Can this be replaced with some call to make_unique and will the []-version of delete be used then?

like image 688
Nordlöw Avatar asked Apr 14 '12 00:04

Nordlöw


1 Answers

Here is another solution (in addition to Mike's):

#include <type_traits>
#include <utility>
#include <memory>

template <class T, class ...Args>
typename std::enable_if
<
    !std::is_array<T>::value,
    std::unique_ptr<T>
>::type
make_unique(Args&& ...args)
{
    return std::unique_ptr<T>(new T(std::forward<Args>(args)...));
}

template <class T>
typename std::enable_if
<
    std::is_array<T>::value,
    std::unique_ptr<T>
>::type
make_unique(std::size_t n)
{
    typedef typename std::remove_extent<T>::type RT;
    return std::unique_ptr<T>(new RT[n]);
}

int main()
{
    auto p1 = make_unique<int>(3);
    auto p2 = make_unique<int[]>(3);
}

Notes:

  1. new T[n] should just default construct n T's.

So make_unique(n) should just default construct n T's.

  1. Issues like this contributed to make_unique not being proposed in C++11. Another issue is: Do we handle custom deleters?

These are not unanswerable questions. But they are questions that haven't yet been fully answered.

like image 71
Howard Hinnant Avatar answered Oct 24 '22 07:10

Howard Hinnant