libpressio version 0.30.1

Bug Fixes:

+ pressio_metrics objects are now default, move, and copy construct able
+ pressio_data objects no longer attempt to copy from empty objects
  which previously caused seg faults

CMakeLists.txt

@@ -1,5 +1,5 @@
 cmake_minimum_required(VERSION 3.13 FATAL_ERROR)
-project(libpressio VERSION "0.30.0" LANGUAGES CXX C)
+project(libpressio VERSION "0.30.1" LANGUAGES CXX C)
 
 #correct was to set a default build type
 # https://blog.kitware.com/cmake-and-the-default-build-type/

COPYRIGHT.txt

@@ -1,6 +1,6 @@
 Copyright © 2020 , UChicago Argonne, LLC
 All Rights Reserved
-[libpressio, Version 0.30.0]
+[libpressio, Version 0.30.1]
 Robert Underwood
 Argonne National Laboratory
 

include/libpressio_ext/cpp/data.h

@@ -217,8 +217,12 @@ struct pressio_data {
   pressio_data& operator=(pressio_data const& rhs) {
     if(this == &rhs) return *this;
     data_dtype = rhs.data_dtype;
-    data_ptr = malloc(rhs.size_in_bytes());
-    memcpy(data_ptr, rhs.data_ptr, rhs.size_in_bytes());
+    if(rhs.has_data()) {
+      data_ptr = malloc(rhs.size_in_bytes());
+      memcpy(data_ptr, rhs.data_ptr, rhs.size_in_bytes());
+    } else {
+      data_ptr = nullptr;
+    }
     metadata_ptr = nullptr;
     deleter = pressio_data_libc_free_fn;
     dims = rhs.dims;
@@ -230,12 +234,14 @@ struct pressio_data {
    * */
   pressio_data(pressio_data const& rhs): 
     data_dtype(rhs.data_dtype),
-    data_ptr(malloc(rhs.size_in_bytes())),
+    data_ptr(rhs.has_data()? malloc(rhs.size_in_bytes()) : nullptr),
     metadata_ptr(nullptr),
     deleter(pressio_data_libc_free_fn),
     dims(rhs.dims)
   {
-    memcpy(data_ptr, rhs.data_ptr, rhs.size_in_bytes());
+    if(rhs.has_data()) {
+      memcpy(data_ptr, rhs.data_ptr, rhs.size_in_bytes());
+    }
   }
   /**
    * move-constructor

include/libpressio_ext/cpp/metrics.h

@@ -113,6 +113,26 @@ struct pressio_metrics {
   /** construct a metrics wrapper*/
   pressio_metrics(std::unique_ptr<libpressio_metrics_plugin>&& metrics): plugin(std::move(metrics)) {}
 
+  /** allow default construction*/
+  pressio_metrics()=default;
+  /**
+   * move construct a metric from another pointer
+   */
+  pressio_metrics(pressio_metrics const& metrics)=default;
+  /**
+   * move assigns a metric from another pointer
+   */
+  pressio_metrics& operator=(pressio_metrics const& metrics)=default;
+  /**
+   * move construct a metric from another pointer
+   */
+  pressio_metrics(pressio_metrics&& metrics)=default;
+  /**
+   * move assigns a metric from another pointer
+   */
+  pressio_metrics& operator=(pressio_metrics&& metrics)=default;
+
+
   /** allow access to underlying plugin*/
   libpressio_metrics_plugin* operator->() const noexcept {return plugin.get();}
   /** allow access to underlying plugin*/