Remove SlotData

extras/conf_test/esp8266.cpp

@@ -10,7 +10,7 @@ static_assert(ARDUINOJSON_LITTLE_ENDIAN == 1, "ARDUINOJSON_LITTLE_ENDIAN");
 
 static_assert(ARDUINOJSON_USE_DOUBLE == 1, "ARDUINOJSON_USE_DOUBLE");
 
-static_assert(sizeof(ArduinoJson::detail::VariantData) == 8, "slot size");
+static_assert(sizeof(VariantData) == 8, "slot size");
 
 void setup() {}
 void loop() {}

src/ArduinoJson/Array/JsonArray.hpp

@@ -23,7 +23,7 @@ class JsonArray : public detail::VariantOperators<JsonArray> {
   JsonArray() : data_(0), resources_(0) {}
 
   // INTERNAL USE ONLY
-  JsonArray(detail::VariantData* data, detail::ResourceManager* resources)
+  JsonArray(detail::ArrayData* data, detail::ResourceManager* resources)
       : data_(data), resources_(resources) {}
 
   // Returns a JsonVariant pointing to the array.
@@ -37,7 +37,7 @@ class JsonArray : public detail::VariantOperators<JsonArray> {
   // Returns a read-only reference to the array.
   // https://arduinojson.org/v7/api/jsonarrayconst/
   operator JsonArrayConst() const {
-    return JsonArrayConst(getData(), resources_);
+    return JsonArrayConst(data_, resources_);
   }
 
   // Appends a new (empty) element to the array.
@@ -55,7 +55,7 @@ class JsonArray : public detail::VariantOperators<JsonArray> {
   template <typename T, detail::enable_if_t<
                             detail::is_same<T, JsonVariant>::value, int> = 0>
   JsonVariant add() const {
-    return JsonVariant(detail::VariantData::addElement(data_, resources_),
+    return JsonVariant(detail::ArrayData::addElement(data_, resources_),
                        resources_);
   }
 
@@ -63,7 +63,7 @@ class JsonArray : public detail::VariantOperators<JsonArray> {
   // https://arduinojson.org/v7/api/jsonarray/add/
   template <typename T>
   bool add(const T& value) const {
-    return detail::VariantData::addValue(data_, value, resources_);
+    return detail::ArrayData::addValue(data_, value, resources_);
   }
 
   // Appends a value to the array.
@@ -71,16 +71,15 @@ class JsonArray : public detail::VariantOperators<JsonArray> {
   template <typename T,
             detail::enable_if_t<!detail::is_const<T>::value, int> = 0>
   bool add(T* value) const {
-    return detail::VariantData::addValue(data_, value, resources_);
+    return detail::ArrayData::addValue(data_, value, resources_);
   }
 
   // Returns an iterator to the first element of the array.
   // https://arduinojson.org/v7/api/jsonarray/begin/
   iterator begin() const {
-    auto array = detail::VariantData::asArray(data_);
+    if (!data_)
-    if (!array)
       return iterator();
-    return iterator(array->createIterator(resources_), resources_);
+    return iterator(data_->createIterator(resources_), resources_);
   }
 
   // Returns an iterator following the last element of the array.
@@ -107,14 +106,13 @@ class JsonArray : public detail::VariantOperators<JsonArray> {
   // Removes the element at the specified iterator.
   // https://arduinojson.org/v7/api/jsonarray/remove/
   void remove(iterator it) const {
-    detail::ArrayData::remove(detail::VariantData::asArray(data_), it.iterator_,
+    detail::ArrayData::remove(data_, it.iterator_, resources_);
-                              resources_);
   }
 
   // Removes the element at the specified index.
   // https://arduinojson.org/v7/api/jsonarray/remove/
   void remove(size_t index) const {
-    detail::VariantData::removeElement(data_, index, resources_);
+    detail::ArrayData::removeElement(data_, index, resources_);
   }
 
   // Removes the element at the specified index.
@@ -129,7 +127,7 @@ class JsonArray : public detail::VariantOperators<JsonArray> {
   // Removes all the elements of the array.
   // https://arduinojson.org/v7/api/jsonarray/clear/
   void clear() const {
-    detail::ArrayData::clear(detail::VariantData::asArray(data_), resources_);
+    detail::ArrayData::clear(data_, resources_);
   }
 
   // Gets or sets the element at the specified index.
@@ -152,25 +150,25 @@ class JsonArray : public detail::VariantOperators<JsonArray> {
   }
 
   operator JsonVariantConst() const {
-    return JsonVariantConst(data_, resources_);
+    return JsonVariantConst(collectionToVariant(data_), resources_);
   }
 
   // Returns true if the reference is unbound.
   // https://arduinojson.org/v7/api/jsonarray/isnull/
   bool isNull() const {
-    return !data_ || !data_->isArray();
+    return data_ == 0;
   }
 
   // Returns true if the reference is bound.
   // https://arduinojson.org/v7/api/jsonarray/isnull/
   operator bool() const {
-    return !isNull();
+    return data_ != 0;
   }
 
   // Returns the depth (nesting level) of the array.
   // https://arduinojson.org/v7/api/jsonarray/nesting/
   size_t nesting() const {
-    return detail::VariantData::nesting(data_, resources_);
+    return detail::VariantData::nesting(collectionToVariant(data_), resources_);
   }
 
   // Returns the number of elements in the array.
@@ -207,14 +205,14 @@ class JsonArray : public detail::VariantOperators<JsonArray> {
   }
 
   detail::VariantData* getData() const {
-    return data_;
+    return collectionToVariant(data_);
   }
 
   detail::VariantData* getOrCreateData() const {
-    return data_;
+    return collectionToVariant(data_);
   }
 
-  detail::VariantData* data_;
+  detail::ArrayData* data_;
   detail::ResourceManager* resources_;
 };
 

src/ArduinoJson/Array/JsonArrayConst.hpp

@@ -24,10 +24,9 @@ class JsonArrayConst : public detail::VariantOperators<JsonArrayConst> {
   // Returns an iterator to the first element of the array.
   // https://arduinojson.org/v7/api/jsonarrayconst/begin/
   iterator begin() const {
-    auto array = detail::VariantData::asArray(data_);
+    if (!data_)
-    if (!array)
       return iterator();
-    return iterator(array->createIterator(resources_), resources_);
+    return iterator(data_->createIterator(resources_), resources_);
   }
 
   // Returns an iterator to the element following the last element of the array.
@@ -40,7 +39,7 @@ class JsonArrayConst : public detail::VariantOperators<JsonArrayConst> {
   JsonArrayConst() : data_(0), resources_(0) {}
 
   // INTERNAL USE ONLY
-  JsonArrayConst(const detail::VariantData* data,
+  JsonArrayConst(const detail::ArrayData* data,
                  const detail::ResourceManager* resources)
       : data_(data), resources_(resources) {}
 
@@ -50,7 +49,7 @@ class JsonArrayConst : public detail::VariantOperators<JsonArrayConst> {
             detail::enable_if_t<detail::is_integral<T>::value, int> = 0>
   JsonVariantConst operator[](T index) const {
     return JsonVariantConst(
-        detail::VariantData::getElement(data_, size_t(index), resources_),
+        detail::ArrayData::getElement(data_, size_t(index), resources_),
         resources_);
   }
 
@@ -72,13 +71,13 @@ class JsonArrayConst : public detail::VariantOperators<JsonArrayConst> {
   // Returns true if the reference is unbound.
   // https://arduinojson.org/v7/api/jsonarrayconst/isnull/
   bool isNull() const {
-    return !data_ || !data_->isArray();
+    return data_ == 0;
   }
 
   // Returns true if the reference is bound.
   // https://arduinojson.org/v7/api/jsonarrayconst/isnull/
   operator bool() const {
-    return !isNull();
+    return data_ != 0;
   }
 
   // Returns the depth (nesting level) of the array.
@@ -101,10 +100,10 @@ class JsonArrayConst : public detail::VariantOperators<JsonArrayConst> {
 
  private:
   const detail::VariantData* getData() const {
-    return data_;
+    return collectionToVariant(data_);
   }
 
-  const detail::VariantData* data_;
+  const detail::ArrayData* data_;
   const detail::ResourceManager* resources_;
 };
 

src/ArduinoJson/Object/JsonObject.hpp

@@ -23,7 +23,7 @@ class JsonObject : public detail::VariantOperators<JsonObject> {
   JsonObject() : data_(0), resources_(0) {}
 
   // INTERNAL USE ONLY
-  JsonObject(detail::VariantData* data, detail::ResourceManager* resource)
+  JsonObject(detail::ObjectData* data, detail::ResourceManager* resource)
       : data_(data), resources_(resource) {}
 
   operator JsonVariant() const {
@@ -37,25 +37,25 @@ class JsonObject : public detail::VariantOperators<JsonObject> {
   }
 
   operator JsonVariantConst() const {
-    return JsonVariantConst(data_, resources_);
+    return JsonVariantConst(collectionToVariant(data_), resources_);
   }
 
   // Returns true if the reference is unbound.
   // https://arduinojson.org/v7/api/jsonobject/isnull/
   bool isNull() const {
-    return !data_ || !data_->isObject();
+    return data_ == 0;
   }
 
   // Returns true if the reference is bound.
   // https://arduinojson.org/v7/api/jsonobject/isnull/
   operator bool() const {
-    return !isNull();
+    return data_ != 0;
   }
 
   // Returns the depth (nesting level) of the object.
   // https://arduinojson.org/v7/api/jsonobject/nesting/
   size_t nesting() const {
-    return detail::VariantData::nesting(data_, resources_);
+    return detail::VariantData::nesting(collectionToVariant(data_), resources_);
   }
 
   // Returns the number of members in the object.
@@ -67,10 +67,9 @@ class JsonObject : public detail::VariantOperators<JsonObject> {
   // Returns an iterator to the first key-value pair of the object.
   // https://arduinojson.org/v7/api/jsonobject/begin/
   iterator begin() const {
-    auto obj = detail::VariantData::asObject(data_);
+    if (!data_)
-    if (!obj)
       return iterator();
-    return iterator(obj->createIterator(resources_), resources_);
+    return iterator(data_->createIterator(resources_), resources_);
   }
 
   // Returns an iterator following the last key-value pair of the object.
@@ -82,7 +81,7 @@ class JsonObject : public detail::VariantOperators<JsonObject> {
   // Removes all the members of the object.
   // https://arduinojson.org/v7/api/jsonobject/clear/
   void clear() const {
-    detail::ObjectData::clear(detail::VariantData::asObject(data_), resources_);
+    detail::ObjectData::clear(data_, resources_);
   }
 
   // Copies an object.
@@ -132,8 +131,7 @@ class JsonObject : public detail::VariantOperators<JsonObject> {
   // Removes the member at the specified iterator.
   // https://arduinojson.org/v7/api/jsonobject/remove/
   FORCE_INLINE void remove(iterator it) const {
-    detail::ObjectData::remove(detail::VariantData::asObject(data_),
+    detail::ObjectData::remove(data_, it.iterator_, resources_);
-                               it.iterator_, resources_);
   }
 
   // Removes the member with the specified key.
@@ -141,7 +139,7 @@ class JsonObject : public detail::VariantOperators<JsonObject> {
   template <typename TString,
             detail::enable_if_t<detail::IsString<TString>::value, int> = 0>
   void remove(const TString& key) const {
-    detail::VariantData::removeMember(data_, detail::adaptString(key),
+    detail::ObjectData::removeMember(data_, detail::adaptString(key),
                                      resources_);
   }
 
@@ -158,7 +156,7 @@ class JsonObject : public detail::VariantOperators<JsonObject> {
   // https://arduinojson.org/v7/api/jsonobject/remove/
   template <typename TChar>
   FORCE_INLINE void remove(TChar* key) const {
-    detail::VariantData::removeMember(data_, detail::adaptString(key),
+    detail::ObjectData::removeMember(data_, detail::adaptString(key),
                                      resources_);
   }
 
@@ -168,7 +166,7 @@ class JsonObject : public detail::VariantOperators<JsonObject> {
             detail::enable_if_t<detail::IsString<TString>::value, int> = 0>
   ARDUINOJSON_DEPRECATED("use obj[key].is<T>() instead")
   bool containsKey(const TString& key) const {
-    return detail::VariantData::getMember(data_, detail::adaptString(key),
+    return detail::ObjectData::getMember(data_, detail::adaptString(key),
                                          resources_) != 0;
   }
 
@@ -180,7 +178,7 @@ class JsonObject : public detail::VariantOperators<JsonObject> {
                                 int> = 0>
   ARDUINOJSON_DEPRECATED("use obj[\"key\"].is<T>() instead")
   bool containsKey(TChar* key) const {
-    return detail::VariantData::getMember(data_, detail::adaptString(key),
+    return detail::ObjectData::getMember(data_, detail::adaptString(key),
                                          resources_) != 0;
   }
 
@@ -233,14 +231,14 @@ class JsonObject : public detail::VariantOperators<JsonObject> {
   }
 
   detail::VariantData* getData() const {
-    return data_;
+    return detail::collectionToVariant(data_);
   }
 
   detail::VariantData* getOrCreateData() const {
-    return data_;
+    return detail::collectionToVariant(data_);
   }
 
-  detail::VariantData* data_;
+  detail::ObjectData* data_;
   detail::ResourceManager* resources_;
 };
 

src/ArduinoJson/Object/JsonObjectConst.hpp

@@ -22,7 +22,7 @@ class JsonObjectConst : public detail::VariantOperators<JsonObjectConst> {
   JsonObjectConst() : data_(0), resources_(0) {}
 
   // INTERNAL USE ONLY
-  JsonObjectConst(const detail::VariantData* data,
+  JsonObjectConst(const detail::ObjectData* data,
                   const detail::ResourceManager* resources)
       : data_(data), resources_(resources) {}
 
@@ -33,13 +33,13 @@ class JsonObjectConst : public detail::VariantOperators<JsonObjectConst> {
   // Returns true if the reference is unbound.
   // https://arduinojson.org/v7/api/jsonobjectconst/isnull/
   bool isNull() const {
-    return !data_ || !data_->isObject();
+    return data_ == 0;
   }
 
   // Returns true if the reference is bound.
   // https://arduinojson.org/v7/api/jsonobjectconst/isnull/
   operator bool() const {
-    return !isNull();
+    return data_ != 0;
   }
 
   // Returns the depth (nesting level) of the object.
@@ -57,10 +57,9 @@ class JsonObjectConst : public detail::VariantOperators<JsonObjectConst> {
   // Returns an iterator to the first key-value pair of the object.
   // https://arduinojson.org/v7/api/jsonobjectconst/begin/
   iterator begin() const {
-    auto obj = detail::VariantData::asObject(data_);
+    if (!data_)
-    if (!obj)
       return iterator();
-    return iterator(obj->createIterator(resources_), resources_);
+    return iterator(data_->createIterator(resources_), resources_);
   }
 
   // Returns an iterator following the last key-value pair of the object.
@@ -75,7 +74,7 @@ class JsonObjectConst : public detail::VariantOperators<JsonObjectConst> {
             detail::enable_if_t<detail::IsString<TString>::value, int> = 0>
   ARDUINOJSON_DEPRECATED("use obj[key].is<T>() instead")
   bool containsKey(const TString& key) const {
-    return detail::VariantData::getMember(data_, detail::adaptString(key),
+    return detail::ObjectData::getMember(data_, detail::adaptString(key),
                                          resources_) != 0;
   }
 
@@ -84,7 +83,7 @@ class JsonObjectConst : public detail::VariantOperators<JsonObjectConst> {
   template <typename TChar>
   ARDUINOJSON_DEPRECATED("use obj[\"key\"].is<T>() instead")
   bool containsKey(TChar* key) const {
-    return detail::VariantData::getMember(data_, detail::adaptString(key),
+    return detail::ObjectData::getMember(data_, detail::adaptString(key),
                                          resources_) != 0;
   }
 
@@ -102,7 +101,7 @@ class JsonObjectConst : public detail::VariantOperators<JsonObjectConst> {
   template <typename TString,
             detail::enable_if_t<detail::IsString<TString>::value, int> = 0>
   JsonVariantConst operator[](const TString& key) const {
-    return JsonVariantConst(detail::VariantData::getMember(
+    return JsonVariantConst(detail::ObjectData::getMember(
                                 data_, detail::adaptString(key), resources_),
                             resources_);
   }
@@ -114,7 +113,7 @@ class JsonObjectConst : public detail::VariantOperators<JsonObjectConst> {
                                     !detail::is_const<TChar>::value,
                                 int> = 0>
   JsonVariantConst operator[](TChar* key) const {
-    return JsonVariantConst(detail::VariantData::getMember(
+    return JsonVariantConst(detail::ObjectData::getMember(
                                 data_, detail::adaptString(key), resources_),
                             resources_);
   }
@@ -138,10 +137,10 @@ class JsonObjectConst : public detail::VariantOperators<JsonObjectConst> {
 
  private:
   const detail::VariantData* getData() const {
-    return data_;
+    return collectionToVariant(data_);
   }
 
-  const detail::VariantData* data_;
+  const detail::ObjectData* data_;
   const detail::ResourceManager* resources_;
 };
 

src/ArduinoJson/Variant/ConverterImpl.hpp

@@ -329,7 +329,9 @@ struct Converter<JsonArrayConst> : private detail::VariantAttorney {
   }
 
   static JsonArrayConst fromJson(JsonVariantConst src) {
-    return JsonArrayConst(getData(src), getResourceManager(src));
+    auto data = getData(src);
+    auto array = data ? data->asArray() : nullptr;
+    return JsonArrayConst(array, getResourceManager(src));
   }
 
   static bool checkJson(JsonVariantConst src) {
@@ -348,7 +350,9 @@ struct Converter<JsonArray> : private detail::VariantAttorney {
   }
 
   static JsonArray fromJson(JsonVariant src) {
-    return JsonArray(getData(src), getResourceManager(src));
+    auto data = getData(src);
+    auto resources = getResourceManager(src);
+    return JsonArray(data != 0 ? data->asArray() : 0, resources);
   }
 
   static bool checkJson(JsonVariant src) {
@@ -367,7 +371,9 @@ struct Converter<JsonObjectConst> : private detail::VariantAttorney {
   }
 
   static JsonObjectConst fromJson(JsonVariantConst src) {
-    return JsonObjectConst(getData(src), getResourceManager(src));
+    auto data = getData(src);
+    auto object = data != 0 ? data->asObject() : nullptr;
+    return JsonObjectConst(object, getResourceManager(src));
   }
 
   static bool checkJson(JsonVariantConst src) {
@@ -386,7 +392,9 @@ struct Converter<JsonObject> : private detail::VariantAttorney {
   }
 
   static JsonObject fromJson(JsonVariant src) {
-    return JsonObject(getData(src), getResourceManager(src));
+    auto data = getData(src);
+    auto resources = getResourceManager(src);
+    return JsonObject(data != 0 ? data->asObject() : 0, resources);
   }
 
   static bool checkJson(JsonVariant src) {

src/ArduinoJson/Variant/JsonVariantVisitor.hpp

@@ -30,13 +30,11 @@ class VisitorAdapter {
       : visitor_(&visitor), resources_(resources) {}
 
   result_type visit(const ArrayData& value) {
-    return visitor_->visit(
+    return visitor_->visit(JsonArrayConst(&value, resources_));
-        JsonArrayConst(collectionToVariant(&value), resources_));
   }
 
   result_type visit(const ObjectData& value) {
-    return visitor_->visit(
+    return visitor_->visit(JsonObjectConst(&value, resources_));
-        JsonObjectConst(collectionToVariant(&value), resources_));
   }
 
   template <typename T>

src/ArduinoJson/Variant/VariantContent.hpp

@@ -16,9 +16,7 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE
 enum class VariantTypeBits : uint8_t {
   OwnedStringBit = 0x01,  // 0000 0001
   NumberBit = 0x08,       // 0000 1000
-#if ARDUINOJSON_USE_8_BYTE_POOL
   EightByteBit = 0x10,    // 0001 0000
-#endif
   CollectionMask = 0x60,
 };
 

src/ArduinoJson/Variant/VariantData.hpp

@@ -182,14 +182,6 @@ class VariantData {
     return const_cast<VariantData*>(this)->asArray();
   }
 
-  static ArrayData* asArray(VariantData* var) {
-    return var ? var->asArray() : 0;
-  }
-
-  static const ArrayData* asArray(const VariantData* var) {
-    return var ? var->asArray() : 0;
-  }
-
   CollectionData* asCollection() {
     return isCollection() ? &content_.asCollection : 0;
   }
@@ -296,14 +288,6 @@ class VariantData {
     return const_cast<VariantData*>(this)->asObject();
   }
 
-  static ObjectData* asObject(VariantData* var) {
-    return var ? var->asObject() : 0;
-  }
-
-  static const ObjectData* asObject(const VariantData* var) {
-    return var ? var->asObject() : 0;
-  }
-
   JsonString asRawString() const {
     switch (type_) {
       case VariantType::RawString:
@@ -584,12 +568,11 @@ class VariantData {
     return content_.asArray;
   }
 
-  static VariantData* toArray(VariantData* var, ResourceManager* resources) {
+  static ArrayData* toArray(VariantData* var, ResourceManager* resources) {
     if (!var)
       return 0;
     var->clear(resources);
-    var->toArray();
+    return &var->toArray();
-    return var;
   }
 
   ObjectData& toObject() {
@@ -599,12 +582,11 @@ class VariantData {
     return content_.asObject;
   }
 
-  static VariantData* toObject(VariantData* var, ResourceManager* resources) {
+  static ObjectData* toObject(VariantData* var, ResourceManager* resources) {
     if (!var)
       return 0;
     var->clear(resources);
-    var->toObject();
+    return &var->toObject();
-    return var;
   }
 
   VariantType type() const {