From 3b8384a993aed55b348bf51bb02900b3aa010ef8 Mon Sep 17 00:00:00 2001 From: Mikko Rasa 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 +#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.45.2