Deleted copy/added move constructor.

pkesseli · pkesseli · commit 68e159501c93 · 2017-05-07T23:30:09.000+01:00
Implicit copy constructor leads to double deletion of files. Added move
constructor to allow returning temp_filet.
diff --git a/src/util/tempfile.h b/src/util/tempfile.h
@@ -29,6 +29,11 @@ class temporary_filet
   {
   }
 
+  // Using the copy constructor would delete the file twice.
+  temporary_filet(const temporary_filet &)=delete;
+
+  temporary_filet(temporary_filet &&)=default;
+
   // get the name
   std::string operator()() const
   {

Original file line number	Diff line number	Diff line change
`@@ -29,6 +29,11 @@ class temporary_filet`
`29`	`29`	`{`
`30`	`30`	`}`
`31`	`31`
	`32`	`+ // Using the copy constructor would delete the file twice.`
	`33`	`+ temporary_filet(const temporary_filet &)=delete;`
	`34`	`+`
	`35`	`+ temporary_filet(temporary_filet &&)=default;`
	`36`	`+`
`32`	`37`	`// get the name`
`33`	`38`	`std::string operator()() const`
`34`	`39`	`{`