From 3b8384a993aed55b348bf51bb02900b3aa010ef8 Mon Sep 17 00:00:00 2001
From: Mikko Rasa <tdb@tdb.fi>
Date: Sat, 11 Jun 2011 14:41:09 +0300
Subject: [PATCH] Add a pimpl Handle class

---
 source/io/handle.cpp       | 53 ++++++++++++++++++++++++++++++++++++++
 source/io/handle.h         | 33 ++++++++++++++++++++++++
 source/io/handle_private.h | 31 ++++++++++++++++++++++
 3 files changed, 117 insertions(+)
 create mode 100644 source/io/handle.cpp
 create mode 100644 source/io/handle.h
 create mode 100644 source/io/handle_private.h

diff --git a/source/io/handle.cpp b/source/io/handle.cpp
new file mode 100644
index 0000000..d5183e5
--- /dev/null
+++ b/source/io/handle.cpp
@@ -0,0 +1,53 @@
+#include "handle.h"
+#include "handle_private.h"
+
+namespace Msp {
+namespace IO {
+
+Handle::Handle():
+	priv(new Private)
+{ }
+
+Handle::Handle(const Handle &other):
+	priv(new Private)
+{
+	priv->handle = other.priv->handle;
+}
+
+Handle &Handle::operator=(const Handle &other)
+{
+	priv->handle = other.priv->handle;
+	return *this;
+}
+
+Handle::~Handle()
+{
+	delete priv;
+}
+
+Handle::operator const void *() const
+{
+#ifdef WIN32
+	return priv->handle!=INVALID_HANDLE_VALUE ? this : 0;
+#else
+	return priv->handle!=-1 ? this : 0;
+#endif
+}
+
+
+Handle::Private::Private():
+#ifdef WIN32
+	handle(INVALID_HANDLE_VALUE)
+#else
+	handle(-1)
+#endif
+{ }
+
+Handle::Private &Handle::Private::operator=(H h)
+{
+	handle = h;
+	return *this;
+}
+
+} // namespace IO
+} // namespace Msp
diff --git a/source/io/handle.h b/source/io/handle.h
new file mode 100644
index 0000000..44b5937
--- /dev/null
+++ b/source/io/handle.h
@@ -0,0 +1,33 @@
+#ifndef MSP_IO_HANDLE_H_
+#define MSP_IO_HANDLE_H_
+
+namespace Msp {
+namespace IO {
+
+class Handle
+{
+public:
+	struct Private;
+
+private:
+	Private *priv;
+
+public:
+	Handle();
+	Handle(const Handle &);
+	Handle &operator=(const Handle &);
+	~Handle();
+
+	Private &operator*() { return *priv; }
+	const Private &operator*() const { return *priv; }
+
+	/** This is effectively a boolean conversion, but avoids accidental
+	conversion to OS native handles.  Unix-based systems use int and win32 uses
+	void *; const void * is not implicitly convertible to either. */
+	operator const void *() const;
+};
+
+} // namespace IO
+} // namespace Msp
+
+#endif
diff --git a/source/io/handle_private.h b/source/io/handle_private.h
new file mode 100644
index 0000000..d73a2b8
--- /dev/null
+++ b/source/io/handle_private.h
@@ -0,0 +1,31 @@
+#ifndef MSP_IO_HANDLE_PRIVATE_H_
+#define MSP_IO_HANDLE_PRIVATE_H_
+
+#ifdef WIN32
+#include <windows.h>
+#endif
+#include "handle.h"
+
+namespace Msp {
+namespace IO {
+
+struct Handle::Private
+{
+#ifdef WIN32
+	typedef HANDLE H;
+#else
+	typedef int H;
+#endif
+
+	H handle;
+
+	Private();
+
+	Private &operator=(H);
+	operator H() const { return handle; }
+};
+
+} // namespace IO
+} // namespace Msp
+
+#endif
-- 
2.43.0