From 2c1b3e5108ed7f6f8ce8311db8aff307e138a2ac Mon Sep 17 00:00:00 2001
From: Niels <niels.lohmann@gmail.com>
Date: Mon, 29 Dec 2014 00:02:13 +0100
Subject: [PATCH] + small cleanup

---
 src/JSON.cc | 80 ++++++++++++++++++++---------------------------------
 1 file changed, 30 insertions(+), 50 deletions(-)

diff --git a/src/JSON.cc b/src/JSON.cc
index c8eb6345..062da5be 100644
--- a/src/JSON.cc
+++ b/src/JSON.cc
@@ -155,9 +155,9 @@ JSON::JSON(list_init_t a) noexcept
     // is a string
     for (const auto& element : a)
     {
-        if (element.type() != value_type::array or
+        if (element._type != value_type::array or
                 element.size() != 2 or
-                element[0].type() != value_type::string)
+                element[0]._type != value_type::string)
         {
 
             // the initializer list describes an array
@@ -173,7 +173,8 @@ JSON::JSON(list_init_t a) noexcept
     for (const JSON& element : a)
     {
         const std::string k = element[0];
-        _value.object->emplace(std::make_pair(std::move(k), std::move(element[1])));
+        _value.object->emplace(std::make_pair(std::move(k),
+                                              std::move(element[1])));
     }
 }
 
@@ -746,9 +747,9 @@ void JSON::push_back(list_init_t a)
     // is a string
     for (const auto& element : a)
     {
-        if (element.type() != value_type::array or
+        if (element._type != value_type::array or
                 element.size() != 2 or
-                element[0].type() != value_type::string)
+                element[0]._type != value_type::string)
         {
             // the initializer list describes an array
             is_array = true;
@@ -1345,23 +1346,16 @@ JSON::const_iterator JSON::cend() const noexcept
 JSON::iterator::iterator(JSON* j) : _object(j)
 {
     if (_object != nullptr)
-        switch (_object->_type)
+    {
+        if (_object->_type == value_type::array)
         {
-            case (value_type::array):
-            {
-                _vi = new array_t::iterator(_object->_value.array->begin());
-                break;
-            }
-            case (value_type::object):
-            {
-                _oi = new object_t::iterator(_object->_value.object->begin());
-                break;
-            }
-            default:
-            {
-                break;
-            }
+            _vi = new array_t::iterator(_object->_value.array->begin());
         }
+        if (_object->_type == value_type::object)
+        {
+            _oi = new object_t::iterator(_object->_value.object->begin());
+        }
+    }
 }
 
 JSON::iterator::iterator(const JSON::iterator& o) : _object(o._object)
@@ -1545,23 +1539,16 @@ JSON& JSON::iterator::value() const
 JSON::const_iterator::const_iterator(const JSON* j) : _object(j)
 {
     if (_object != nullptr)
-        switch (_object->_type)
+    {
+        if (_object->_type == value_type::array)
         {
-            case (value_type::array):
-            {
-                _vi = new array_t::const_iterator(_object->_value.array->begin());
-                break;
-            }
-            case (value_type::object):
-            {
-                _oi = new object_t::const_iterator(_object->_value.object->begin());
-                break;
-            }
-            default:
-            {
-                break;
-            }
+            _vi = new array_t::const_iterator(_object->_value.array->begin());
         }
+        if (_object->_type == value_type::object)
+        {
+            _oi = new object_t::const_iterator(_object->_value.object->begin());
+        }
+    }
 }
 
 JSON::const_iterator::const_iterator(const JSON::const_iterator& o) : _object(o._object)
@@ -1582,23 +1569,16 @@ JSON::const_iterator::const_iterator(const JSON::const_iterator& o) : _object(o.
 JSON::const_iterator::const_iterator(const JSON::iterator& o) : _object(o._object)
 {
     if (_object != nullptr)
-        switch (_object->_type)
+    {
+        if (_object->_type == value_type::array)
         {
-            case (value_type::array):
-            {
-                _vi = new array_t::const_iterator(*(o._vi));
-                break;
-            }
-            case (value_type::object):
-            {
-                _oi = new object_t::const_iterator(*(o._oi));
-                break;
-            }
-            default:
-            {
-                break;
-            }
+            _vi = new array_t::const_iterator(*(o._vi));
         }
+        if (_object->_type == value_type::object)
+        {
+            _oi = new object_t::const_iterator(*(o._oi));
+        }
+    }
 }
 
 JSON::const_iterator::~const_iterator()