From ef04593de1bf4cc370131dd54f89b11637c53024 Mon Sep 17 00:00:00 2001 From: David Blaikie Date: Thu, 21 Feb 2013 00:27:28 +0000 Subject: [PATCH] Provide a "None" value for convenience when using Optional() This implementation of NoneType/None does have some holes but I haven't found one that doesn't - open to improvement. llvm-svn: 175696 --- llvm/include/llvm/ADT/None.h | 27 +++++++++++++++++++++++++++ llvm/include/llvm/ADT/Optional.h | 2 ++ llvm/lib/Support/LockFileManager.cpp | 4 ++-- 3 files changed, 31 insertions(+), 2 deletions(-) create mode 100644 llvm/include/llvm/ADT/None.h diff --git a/llvm/include/llvm/ADT/None.h b/llvm/include/llvm/ADT/None.h new file mode 100644 index 0000000..83c9521 --- /dev/null +++ b/llvm/include/llvm/ADT/None.h @@ -0,0 +1,27 @@ +//===-- None.h - Simple null value for implicit construction ------*- C++ -*-=// +// +// The LLVM Compiler Infrastructure +// +// This file is distributed under the University of Illinois Open Source +// License. See LICENSE.TXT for details. +// +//===----------------------------------------------------------------------===// +// +// This file provides None, an enumerant for use in implicit constructors +// of various (usually templated) types to make such construction more +// terse. +// +//===----------------------------------------------------------------------===// + +#ifndef LLVM_ADT_NONE_H +#define LLVM_ADT_NONE_H + +namespace llvm { +/// \brief A simple null object to allow implicit construction of Optional +/// and similar types without having to spell out the specialization's name. +enum NoneType { + None +}; +} + +#endif diff --git a/llvm/include/llvm/ADT/Optional.h b/llvm/include/llvm/ADT/Optional.h index 6c91a13..b0d09f6 100644 --- a/llvm/include/llvm/ADT/Optional.h +++ b/llvm/include/llvm/ADT/Optional.h @@ -16,6 +16,7 @@ #ifndef LLVM_ADT_OPTIONAL_H #define LLVM_ADT_OPTIONAL_H +#include "llvm/ADT/None.h" #include "llvm/Support/Compiler.h" #include "llvm/Support/AlignOf.h" #include @@ -31,6 +32,7 @@ class Optional { AlignedCharArrayUnion storage; bool hasVal; public: + Optional(NoneType) : hasVal(false) {} explicit Optional() : hasVal(false) {} Optional(const T &y) : hasVal(true) { new (storage.buffer) T(y); diff --git a/llvm/lib/Support/LockFileManager.cpp b/llvm/lib/Support/LockFileManager.cpp index 31eec75..92d8b83 100644 --- a/llvm/lib/Support/LockFileManager.cpp +++ b/llvm/lib/Support/LockFileManager.cpp @@ -31,7 +31,7 @@ LockFileManager::readLockFile(StringRef LockFileName) { // to read, so we just return. bool Exists = false; if (sys::fs::exists(LockFileName, Exists) || !Exists) - return Optional >(); + return None; // Read the owning host and PID out of the lock file. If it appears that the // owning process is dead, the lock file is invalid. @@ -45,7 +45,7 @@ LockFileManager::readLockFile(StringRef LockFileName) { // Delete the lock file. It's invalid anyway. bool Existed; sys::fs::remove(LockFileName, Existed); - return Optional >(); + return None; } bool LockFileManager::processStillExecuting(StringRef Hostname, int PID) { -- 2.7.4