From 9344598eb04fb40947b132fd06468d98d737eb07 Mon Sep 17 00:00:00 2001
From: John Wiegley <johnw@newartisans.com>
Date: Thu, 12 Feb 2009 02:35:22 -0400
Subject: Inlined value_t::_clear(), since it only had one caller.

---
 src/value.h | 26 +++++++++-----------------
 1 file changed, 9 insertions(+), 17 deletions(-)

(limited to 'src')

diff --git a/src/value.h b/src/value.h
index 8e144bb7..02c338f2 100644
--- a/src/value.h
+++ b/src/value.h
@@ -227,30 +227,19 @@ private:
   intrusive_ptr<storage_t> storage;
 
   /**
-   * _dup() makes a private copy of the current value (if necessary)
-   * so it can subsequently be modified.
-   *
-   * _clear() removes our pointer to the current value and initializes
-   * a new storage bin for things to be stored in.
+   * Make a private copy of the current value (if necessary) so it can
+   * subsequently be modified.
    */
   void _dup() {
     assert(storage);
     if (storage->refc > 1)
       storage = new storage_t(*storage.get());
   }
-  void _clear() {
-    if (! storage || storage->refc > 1) {
-      storage = new storage_t;
-    } else {
-      storage->data = false;	// destruct any other type
-      storage->type = VOID;
-    }
-  }
 
   /**
-   * Because boolean "true" and "false" are so common, a pair of
-   * static references are kept to prevent the creation of throwaway
-   * storage_t objects just to represent these two common values.
+   * Because boolean "true" and "false" are so common, a pair of static
+   * references are kept to prevent the creation of throwaway storage_t
+   * objects just to represent these two common values.
    */
   static intrusive_ptr<storage_t> true_value;
   static intrusive_ptr<storage_t> false_value;
@@ -480,7 +469,10 @@ private:
 #endif
       assert(is_null());
     } else {
-      _clear();
+      if (! storage || storage->refc > 1)
+	storage = new storage_t;
+      else
+	storage->data = false;	// destruct all other types
       storage->type = new_type;
       assert(is_type(new_type));
     }
-- 
cgit v1.2.3