Fix build with Clang 10

Clang 10 says “error: explicitly defaulted move constructor is
implicitly deleted [...] note: move constructor of 'TemporaryFile' is
implicitly deleted because base class 'NonCopyable' has a deleted move
constructor”, so we’ll just have to do without NonCopyable. The effect
will be the same in pracice due to the “Fd fd” member not being
copyable.

diff --git a/src/TemporaryFile.hpp b/src/TemporaryFile.hpp
index dc64f8b4fb74299aa5fd6441a1109a1034a42314..d02eafab501798957738a0c886ce32830398fa99 100644 (file)
--- a/src/TemporaryFile.hpp
+++ b/src/TemporaryFile.hpp
@@ -19,13 +19,12 @@
 #pragma once
 
 #include "Fd.hpp"
-#include "NonCopyable.hpp"
 
 #include "third_party/nonstd/string_view.hpp"
 
 // This class represents a unique temporary file created by mkstemp. The file is
 // not deleted by the destructor.
-class TemporaryFile : NonCopyable
+class TemporaryFile
 {
 public:
   // `path_prefix` is the base path. The resulting filename will be this path