event/Event{Pipe,FD}: auto-create in constructor

Errors are fatal now.  This makes the class a lot easier to use.

1 files changed, 3 insertions, 14 deletions

diff --git a/src/event/EventFD.hxx b/src/event/EventFD.hxx
index 803957354..67a0258ab 100644
--- a/src/event/EventFD.hxx
+++ b/src/event/EventFD.hxx
@@ -22,33 +22,22 @@
 
 #include "check.h"
 
-#include <assert.h>
-
 /**
  * A class that wraps eventfd().
  *
- * For optimization purposes, this class does not have a constructor
- * or a destructor.
+ * Errors in the constructor are fatal.
  */
 class EventFD {
 	int fd;
 
 public:
-#ifdef NDEBUG
-	EventFD() = default;
-#else
-	EventFD():fd(-1) {}
-#endif
+	EventFD();
+	~EventFD();
 
 	EventFD(const EventFD &other) = delete;
 	EventFD &operator=(const EventFD &other) = delete;
 
-	bool Create();
-	void Destroy();
-
 	int Get() const {
-		assert(fd >= 0);
-
 		return fd;
 	}