Add copy constructor and assignment operator for XPtr (#330)

NEWS.md

@@ -8,6 +8,8 @@
 
 * `xml_find_all.xml_node()` fails more informatively the `xpath` parameter is the wrong type (@michaelchirico)
 
+* `XPtr` gets explicit copy constructor and assignment operator definitions, which were two missing components of the [Rule of three](https://en.wikipedia.org/wiki/Rule_of_three_(C%2B%2B_programming)) (@michaelchirico)
+
 # xml2 1.3.2
 
 * `read_html()` and `read_xml()` now error if passed strings of length greater than one (#121)

inst/include/xml2_types.h

@@ -21,6 +21,20 @@ template <typename T> class XPtr {
     data_ = R_MakeExternalPtr((void *) p, R_NilValue, R_NilValue);
     R_PreserveObject(data_);
   }
+
+  XPtr(const XPtr<T> &old) {
+    data_ = old.data_;
+    R_PreserveObject(data_);
+  }
+
+  XPtr& operator=(const XPtr<T> &other) {
+    R_PreserveObject(other.data_);
+    if (data_ != NULL) {
+      R_ReleaseObject(data_);
+    }
+    data_ = other.data_;
+    return *this;
+  }
 
   operator SEXP() const { return data_; }