Fixes #713 + minor unique_region tweaks:

* The `reset()` method now passes a pointer, rather than a region, to the deleter
* Now using `get_deleter()` in the destructor as well
* Comment corrections and deletions
* Added an instantiation of `unique_region` to the `new_cpp_standard` example program

Thanks goes to GitHub user @ralwing for catching the bug and proposing the fix.

Author: eyalroz

examples/other/new_cpp_standard/main.cpp

@@ -26,10 +26,20 @@ cuda::device::id_t get_current_device_id()
 
 void unique_spans()
 {
-	cuda::unique_span<float> data1(nullptr, 0, cuda::detail_::default_span_deleter<float>);
-	cuda::unique_span<float> data2(nullptr, 0, cuda::detail_::default_span_deleter<float>);
+	auto device = cuda::device::current::get();
+	auto us1 = cuda::make_unique_span<float>(10);
+	auto us2 = cuda::unique_span<float>(nullptr, 0, cuda::detail_::default_span_deleter<float>);
+	auto us3 = cuda::unique_span<float>(nullptr, 0, cuda::detail_::default_span_deleter<float>);
+	us2 = std::move(us3);
+}
 
-	data1 = std::move(data2);
+void unique_regions()
+{
+	auto device = cuda::device::current::get();
+	auto ur1 = cuda::memory::device::make_unique_region(device, 10);
+	auto ur2 = cuda::memory::device::make_unique_region(device, 10);
+	using std::swap;
+	swap(ur1, ur2);
 }
 
 int main() 

src/cuda/api/unique_region.hpp

@@ -60,7 +60,7 @@ class unique_region : public region_t {
 
     // Note: No constructor which also takes a deleter. We do not hold a deleter
     // member - unlike unique_ptr's. If we wanted a general-purpose unique region
-    // that's not just GPU allcoation-oriented, we might have had one of those.
+    // that's not just GPU allocation-oriented, we might have had one of those.
 
     /// Move constructor.
     unique_region(unique_region&& other) noexcept : unique_region(other.release()) { }
@@ -74,7 +74,7 @@ class unique_region : public region_t {
     ~unique_region() noexcept
     {
         if (data() != nullptr) {
-            deleter_type{}(data());
+            get_deleter()(data());
         }
 		static_cast<region_t&>(*this) = region_t{ nullptr, 0 };
     }
@@ -89,8 +89,6 @@ class unique_region : public region_t {
         return *this;
     }
 
-    // No "assignment from anoterh type", a s
-
     /// Reset the %unique_region to empty, invoking the deleter if necessary.
     unique_region&
     operator=(::std::nullptr_t) noexcept
@@ -111,7 +109,7 @@ class unique_region : public region_t {
 
     /// Return a deleter of the fixed type (it can't be a reference -
     /// we don't keep a deleter object)
-    deleter_type get_deleter() const noexcept { return Deleter{}; }
+    deleter_type get_deleter() const noexcept { return deleter_type{}; }
 
     /// Return @c true if the stored pointer is not null.
     explicit operator bool() const noexcept { return data() != nullptr; }
@@ -138,7 +136,7 @@ class unique_region : public region_t {
     {
         ::std::swap<region_t>(*this, region);
         if (region.start() != nullptr) {
-            get_deleter()(region);
+            get_deleter()(region.data());
         }
     }