diff --git a/CHANGELOG.md b/CHANGELOG.md index 18a824a5..a6fa85ba 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,6 +1,11 @@ ArduinoJson: change log ======================= +HEAD +---- + +* Fix compatibility with the Zephyr Project (issue #1905) + v6.21.1 (2023-03-27) ------- diff --git a/extras/tests/Helpers/CustomReader.hpp b/extras/tests/Helpers/CustomReader.hpp index da19d3fb..66525a25 100644 --- a/extras/tests/Helpers/CustomReader.hpp +++ b/extras/tests/Helpers/CustomReader.hpp @@ -7,18 +7,18 @@ #include class CustomReader { - std::stringstream _stream; + std::stringstream stream_; public: - CustomReader(const char* input) : _stream(input) {} + CustomReader(const char* input) : stream_(input) {} CustomReader(const CustomReader&) = delete; int read() { - return _stream.get(); + return stream_.get(); } size_t readBytes(char* buffer, size_t length) { - _stream.read(buffer, static_cast(length)); - return static_cast(_stream.gcount()); + stream_.read(buffer, static_cast(length)); + return static_cast(stream_.gcount()); } }; diff --git a/extras/tests/JsonDocument/BasicJsonDocument.cpp b/extras/tests/JsonDocument/BasicJsonDocument.cpp index 9f034f07..348fd94c 100644 --- a/extras/tests/JsonDocument/BasicJsonDocument.cpp +++ b/extras/tests/JsonDocument/BasicJsonDocument.cpp @@ -10,29 +10,29 @@ class SpyingAllocator { public: - SpyingAllocator(const SpyingAllocator& src) : _log(src._log) {} - SpyingAllocator(std::ostream& log) : _log(log) {} + SpyingAllocator(const SpyingAllocator& src) : log_(src.log_) {} + SpyingAllocator(std::ostream& log) : log_(log) {} SpyingAllocator& operator=(const SpyingAllocator& src) = delete; void* allocate(size_t n) { - _log << "A" << n; + log_ << "A" << n; return malloc(n); } void deallocate(void* p) { - _log << "F"; + log_ << "F"; free(p); } private: - std::ostream& _log; + std::ostream& log_; }; class ControllableAllocator { public: - ControllableAllocator() : _enabled(true) {} + ControllableAllocator() : enabled_(true) {} void* allocate(size_t n) { - return _enabled ? malloc(n) : 0; + return enabled_ ? malloc(n) : 0; } void deallocate(void* p) { @@ -40,11 +40,11 @@ class ControllableAllocator { } void disable() { - _enabled = false; + enabled_ = false; } private: - bool _enabled; + bool enabled_; }; TEST_CASE("BasicJsonDocument") { diff --git a/extras/tests/JsonDocument/shrinkToFit.cpp b/extras/tests/JsonDocument/shrinkToFit.cpp index cc769725..a4423340 100644 --- a/extras/tests/JsonDocument/shrinkToFit.cpp +++ b/extras/tests/JsonDocument/shrinkToFit.cpp @@ -10,36 +10,36 @@ class ArmoredAllocator { public: - ArmoredAllocator() : _ptr(0), _size(0) {} + ArmoredAllocator() : ptr_(0), size_(0) {} void* allocate(size_t size) { - _ptr = malloc(size); - _size = size; - return _ptr; + ptr_ = malloc(size); + size_ = size; + return ptr_; } void deallocate(void* ptr) { - REQUIRE(ptr == _ptr); + REQUIRE(ptr == ptr_); free(ptr); - _ptr = 0; - _size = 0; + ptr_ = 0; + size_ = 0; } void* reallocate(void* ptr, size_t new_size) { - REQUIRE(ptr == _ptr); + REQUIRE(ptr == ptr_); // don't call realloc, instead alloc a new buffer and erase the old one // this way we make sure we support relocation void* new_ptr = malloc(new_size); - memcpy(new_ptr, _ptr, std::min(new_size, _size)); - memset(_ptr, '#', _size); // erase - free(_ptr); - _ptr = new_ptr; + memcpy(new_ptr, ptr_, std::min(new_size, size_)); + memset(ptr_, '#', size_); // erase + free(ptr_); + ptr_ = new_ptr; return new_ptr; } private: - void* _ptr; - size_t _size; + void* ptr_; + size_t size_; }; typedef BasicJsonDocument ShrinkToFitTestDocument; diff --git a/extras/tests/JsonSerializer/CustomWriter.cpp b/extras/tests/JsonSerializer/CustomWriter.cpp index b6d8be72..a84657a7 100644 --- a/extras/tests/JsonSerializer/CustomWriter.cpp +++ b/extras/tests/JsonSerializer/CustomWriter.cpp @@ -12,21 +12,21 @@ class CustomWriter { CustomWriter& operator=(const CustomWriter&) = delete; size_t write(uint8_t c) { - _str.append(1, static_cast(c)); + str_.append(1, static_cast(c)); return 1; } size_t write(const uint8_t* s, size_t n) { - _str.append(reinterpret_cast(s), n); + str_.append(reinterpret_cast(s), n); return n; } const std::string& str() const { - return _str; + return str_; } private: - std::string _str; + std::string str_; }; TEST_CASE("CustomWriter") { diff --git a/extras/tests/JsonVariant/converters.cpp b/extras/tests/JsonVariant/converters.cpp index 4de066a0..89b34ded 100644 --- a/extras/tests/JsonVariant/converters.cpp +++ b/extras/tests/JsonVariant/converters.cpp @@ -74,18 +74,18 @@ TEST_CASE("Custom converter with overloading") { class Complex { public: - explicit Complex(double r, double i) : _real(r), _imag(i) {} + explicit Complex(double r, double i) : real_(r), imag_(i) {} double real() const { - return _real; + return real_; } double imag() const { - return _imag; + return imag_; } private: - double _real, _imag; + double real_, imag_; }; namespace ArduinoJson { diff --git a/extras/tests/Misc/Readers.cpp b/extras/tests/Misc/Readers.cpp index bd04e16e..0b8f4d50 100644 --- a/extras/tests/Misc/Readers.cpp +++ b/extras/tests/Misc/Readers.cpp @@ -170,19 +170,19 @@ TEST_CASE("IteratorReader") { class StreamStub : public Stream { public: - StreamStub(const char* s) : _stream(s) {} + StreamStub(const char* s) : stream_(s) {} int read() { - return _stream.get(); + return stream_.get(); } size_t readBytes(char* buffer, size_t length) { - _stream.read(buffer, static_cast(length)); - return static_cast(_stream.gcount()); + stream_.read(buffer, static_cast(length)); + return static_cast(stream_.gcount()); } private: - std::istringstream _stream; + std::istringstream stream_; }; TEST_CASE("Reader") { diff --git a/extras/tests/Misc/conflicts.cpp b/extras/tests/Misc/conflicts.cpp index 09566825..2e33ea10 100644 --- a/extras/tests/Misc/conflicts.cpp +++ b/extras/tests/Misc/conflicts.cpp @@ -52,5 +52,8 @@ #define BLOCKSIZE #define CAPACITY +// issue #1905 +#define _current + // catch.hpp mutes several warnings, this file also allows to detect them #include "ArduinoJson.h" diff --git a/extras/tests/Misc/printable.cpp b/extras/tests/Misc/printable.cpp index 0c6e8e19..09c23ad1 100644 --- a/extras/tests/Misc/printable.cpp +++ b/extras/tests/Misc/printable.cpp @@ -29,21 +29,21 @@ struct PrintAllAtOnce { template struct PrintableString : public Printable { - PrintableString(const char* s) : _str(s), _total(0) {} + PrintableString(const char* s) : str_(s), total_(0) {} virtual size_t printTo(Print& p) const { - size_t result = PrintPolicy::printStringTo(_str, p); - _total += result; + size_t result = PrintPolicy::printStringTo(str_, p); + total_ += result; return result; } size_t totalBytesWritten() const { - return _total; + return total_; } private: - std::string _str; - mutable size_t _total; + std::string str_; + mutable size_t total_; }; TEST_CASE("Printable") { diff --git a/src/ArduinoJson/Array/ElementProxy.hpp b/src/ArduinoJson/Array/ElementProxy.hpp index 1a9e9ce2..d6e9aa5d 100644 --- a/src/ArduinoJson/Array/ElementProxy.hpp +++ b/src/ArduinoJson/Array/ElementProxy.hpp @@ -17,10 +17,10 @@ class ElementProxy : public VariantRefBase>, public: ElementProxy(TUpstream upstream, size_t index) - : _upstream(upstream), _index(index) {} + : upstream_(upstream), index_(index) {} ElementProxy(const ElementProxy& src) - : _upstream(src._upstream), _index(src._index) {} + : upstream_(src.upstream_), index_(src.index_) {} FORCE_INLINE ElementProxy& operator=(const ElementProxy& src) { this->set(src); @@ -41,20 +41,20 @@ class ElementProxy : public VariantRefBase>, private: FORCE_INLINE MemoryPool* getPool() const { - return VariantAttorney::getPool(_upstream); + return VariantAttorney::getPool(upstream_); } FORCE_INLINE VariantData* getData() const { - return variantGetElement(VariantAttorney::getData(_upstream), _index); + return variantGetElement(VariantAttorney::getData(upstream_), index_); } FORCE_INLINE VariantData* getOrCreateData() const { - return variantGetOrAddElement(VariantAttorney::getOrCreateData(_upstream), - _index, VariantAttorney::getPool(_upstream)); + return variantGetOrAddElement(VariantAttorney::getOrCreateData(upstream_), + index_, VariantAttorney::getPool(upstream_)); } - TUpstream _upstream; - size_t _index; + TUpstream upstream_; + size_t index_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Array/JsonArray.hpp b/src/ArduinoJson/Array/JsonArray.hpp index 00df67ce..d208e417 100644 --- a/src/ArduinoJson/Array/JsonArray.hpp +++ b/src/ArduinoJson/Array/JsonArray.hpp @@ -20,32 +20,32 @@ class JsonArray : public detail::VariantOperators { typedef JsonArrayIterator iterator; // Constructs an unbound reference. - FORCE_INLINE JsonArray() : _data(0), _pool(0) {} + FORCE_INLINE JsonArray() : data_(0), pool_(0) {} // INTERNAL USE ONLY FORCE_INLINE JsonArray(detail::MemoryPool* pool, detail::CollectionData* data) - : _data(data), _pool(pool) {} + : data_(data), pool_(pool) {} // Returns a JsonVariant pointing to the array. // https://arduinojson.org/v6/api/jsonvariant/ operator JsonVariant() { - void* data = _data; // prevent warning cast-align - return JsonVariant(_pool, reinterpret_cast(data)); + void* data = data_; // prevent warning cast-align + return JsonVariant(pool_, reinterpret_cast(data)); } // Returns a read-only reference to the array. // https://arduinojson.org/v6/api/jsonarrayconst/ operator JsonArrayConst() const { - return JsonArrayConst(_data); + return JsonArrayConst(data_); } // Appends a new (null) element to the array. // Returns a reference to the new element. // https://arduinojson.org/v6/api/jsonarray/add/ JsonVariant add() const { - if (!_data) + if (!data_) return JsonVariant(); - return JsonVariant(_pool, _data->addElement(_pool)); + return JsonVariant(pool_, data_->addElement(pool_)); } // Appends a value to the array. @@ -65,9 +65,9 @@ class JsonArray : public detail::VariantOperators { // Returns an iterator to the first element of the array. // https://arduinojson.org/v6/api/jsonarray/begin/ FORCE_INLINE iterator begin() const { - if (!_data) + if (!data_) return iterator(); - return iterator(_pool, _data->head()); + return iterator(pool_, data_->head()); } // Returns an iterator following the last element of the array. @@ -79,41 +79,41 @@ class JsonArray : public detail::VariantOperators { // Copies an array. // https://arduinojson.org/v6/api/jsonarray/set/ FORCE_INLINE bool set(JsonArrayConst src) const { - if (!_data || !src._data) + if (!data_ || !src.data_) return false; - return _data->copyFrom(*src._data, _pool); + return data_->copyFrom(*src.data_, pool_); } // Compares the content of two arrays. FORCE_INLINE bool operator==(JsonArray rhs) const { - return JsonArrayConst(_data) == JsonArrayConst(rhs._data); + return JsonArrayConst(data_) == JsonArrayConst(rhs.data_); } // Removes the element at the specified iterator. // ⚠️ Doesn't release the memory associated with the removed element. // https://arduinojson.org/v6/api/jsonarray/remove/ FORCE_INLINE void remove(iterator it) const { - if (!_data) + if (!data_) return; - _data->removeSlot(it._slot); + data_->removeSlot(it.slot_); } // Removes the element at the specified index. // ⚠️ Doesn't release the memory associated with the removed element. // https://arduinojson.org/v6/api/jsonarray/remove/ FORCE_INLINE void remove(size_t index) const { - if (!_data) + if (!data_) return; - _data->removeElement(index); + data_->removeElement(index); } // Removes all the elements of the array. // ⚠️ Doesn't release the memory associated with the removed elements. // https://arduinojson.org/v6/api/jsonarray/clear/ void clear() const { - if (!_data) + if (!data_) return; - _data->clear(); + data_->clear(); } // Gets or sets the element at the specified index. @@ -133,54 +133,54 @@ class JsonArray : public detail::VariantOperators { } operator JsonVariantConst() const { - return JsonVariantConst(collectionToVariant(_data)); + return JsonVariantConst(collectionToVariant(data_)); } // Returns true if the reference is unbound. // https://arduinojson.org/v6/api/jsonarray/isnull/ FORCE_INLINE bool isNull() const { - return _data == 0; + return data_ == 0; } // Returns true if the reference is bound. // https://arduinojson.org/v6/api/jsonarray/isnull/ FORCE_INLINE operator bool() const { - return _data != 0; + return data_ != 0; } // Returns the number of bytes occupied by the array. // https://arduinojson.org/v6/api/jsonarray/memoryusage/ FORCE_INLINE size_t memoryUsage() const { - return _data ? _data->memoryUsage() : 0; + return data_ ? data_->memoryUsage() : 0; } // Returns the depth (nesting level) of the array. // https://arduinojson.org/v6/api/jsonarray/nesting/ FORCE_INLINE size_t nesting() const { - return variantNesting(collectionToVariant(_data)); + return variantNesting(collectionToVariant(data_)); } // Returns the number of elements in the array. // https://arduinojson.org/v6/api/jsonarray/size/ FORCE_INLINE size_t size() const { - return _data ? _data->size() : 0; + return data_ ? data_->size() : 0; } private: detail::MemoryPool* getPool() const { - return _pool; + return pool_; } detail::VariantData* getData() const { - return collectionToVariant(_data); + return collectionToVariant(data_); } detail::VariantData* getOrCreateData() const { - return collectionToVariant(_data); + return collectionToVariant(data_); } - detail::CollectionData* _data; - detail::MemoryPool* _pool; + detail::CollectionData* data_; + detail::MemoryPool* pool_; }; template <> diff --git a/src/ArduinoJson/Array/JsonArrayConst.hpp b/src/ArduinoJson/Array/JsonArrayConst.hpp index 6c60e981..6a6463c8 100644 --- a/src/ArduinoJson/Array/JsonArrayConst.hpp +++ b/src/ArduinoJson/Array/JsonArrayConst.hpp @@ -24,9 +24,9 @@ class JsonArrayConst : public detail::VariantOperators { // Returns an iterator to the first element of the array. // https://arduinojson.org/v6/api/jsonarrayconst/begin/ FORCE_INLINE iterator begin() const { - if (!_data) + if (!data_) return iterator(); - return iterator(_data->head()); + return iterator(data_->head()); } // Returns an iterator to the element following the last element of the array. @@ -36,18 +36,18 @@ class JsonArrayConst : public detail::VariantOperators { } // Creates an unbound reference. - FORCE_INLINE JsonArrayConst() : _data(0) {} + FORCE_INLINE JsonArrayConst() : data_(0) {} // INTERNAL USE ONLY FORCE_INLINE JsonArrayConst(const detail::CollectionData* data) - : _data(data) {} + : data_(data) {} // Compares the content of two arrays. // Returns true if the two arrays are equal. FORCE_INLINE bool operator==(JsonArrayConst rhs) const { - if (_data == rhs._data) + if (data_ == rhs.data_) return true; - if (!_data || !rhs._data) + if (!data_ || !rhs.data_) return false; iterator it1 = begin(); @@ -70,49 +70,49 @@ class JsonArrayConst : public detail::VariantOperators { // Returns the element at the specified index. // https://arduinojson.org/v6/api/jsonarrayconst/subscript/ FORCE_INLINE JsonVariantConst operator[](size_t index) const { - return JsonVariantConst(_data ? _data->getElement(index) : 0); + return JsonVariantConst(data_ ? data_->getElement(index) : 0); } operator JsonVariantConst() const { - return JsonVariantConst(collectionToVariant(_data)); + return JsonVariantConst(collectionToVariant(data_)); } // Returns true if the reference is unbound. // https://arduinojson.org/v6/api/jsonarrayconst/isnull/ FORCE_INLINE bool isNull() const { - return _data == 0; + return data_ == 0; } // Returns true if the reference is bound. // https://arduinojson.org/v6/api/jsonarrayconst/isnull/ FORCE_INLINE operator bool() const { - return _data != 0; + return data_ != 0; } // Returns the number of bytes occupied by the array. // https://arduinojson.org/v6/api/jsonarrayconst/memoryusage/ FORCE_INLINE size_t memoryUsage() const { - return _data ? _data->memoryUsage() : 0; + return data_ ? data_->memoryUsage() : 0; } // Returns the depth (nesting level) of the array. // https://arduinojson.org/v6/api/jsonarrayconst/nesting/ FORCE_INLINE size_t nesting() const { - return variantNesting(collectionToVariant(_data)); + return variantNesting(collectionToVariant(data_)); } // Returns the number of elements in the array. // https://arduinojson.org/v6/api/jsonarrayconst/size/ FORCE_INLINE size_t size() const { - return _data ? _data->size() : 0; + return data_ ? data_->size() : 0; } private: const detail::VariantData* getData() const { - return collectionToVariant(_data); + return collectionToVariant(data_); } - const detail::CollectionData* _data; + const detail::CollectionData* data_; }; template <> diff --git a/src/ArduinoJson/Array/JsonArrayIterator.hpp b/src/ArduinoJson/Array/JsonArrayIterator.hpp index 3e1b7526..d9048b28 100644 --- a/src/ArduinoJson/Array/JsonArrayIterator.hpp +++ b/src/ArduinoJson/Array/JsonArrayIterator.hpp @@ -12,110 +12,110 @@ ARDUINOJSON_BEGIN_PUBLIC_NAMESPACE class VariantPtr { public: VariantPtr(detail::MemoryPool* pool, detail::VariantData* data) - : _variant(pool, data) {} + : variant_(pool, data) {} JsonVariant* operator->() { - return &_variant; + return &variant_; } JsonVariant& operator*() { - return _variant; + return variant_; } private: - JsonVariant _variant; + JsonVariant variant_; }; class JsonArrayIterator { friend class JsonArray; public: - JsonArrayIterator() : _slot(0) {} + JsonArrayIterator() : slot_(0) {} explicit JsonArrayIterator(detail::MemoryPool* pool, detail::VariantSlot* slot) - : _pool(pool), _slot(slot) {} + : pool_(pool), slot_(slot) {} JsonVariant operator*() const { - return JsonVariant(_pool, _slot->data()); + return JsonVariant(pool_, slot_->data()); } VariantPtr operator->() { - return VariantPtr(_pool, _slot->data()); + return VariantPtr(pool_, slot_->data()); } bool operator==(const JsonArrayIterator& other) const { - return _slot == other._slot; + return slot_ == other.slot_; } bool operator!=(const JsonArrayIterator& other) const { - return _slot != other._slot; + return slot_ != other.slot_; } JsonArrayIterator& operator++() { - _slot = _slot->next(); + slot_ = slot_->next(); return *this; } JsonArrayIterator& operator+=(size_t distance) { - _slot = _slot->next(distance); + slot_ = slot_->next(distance); return *this; } private: - detail::MemoryPool* _pool; - detail::VariantSlot* _slot; + detail::MemoryPool* pool_; + detail::VariantSlot* slot_; }; class VariantConstPtr { public: - VariantConstPtr(const detail::VariantData* data) : _variant(data) {} + VariantConstPtr(const detail::VariantData* data) : variant_(data) {} JsonVariantConst* operator->() { - return &_variant; + return &variant_; } JsonVariantConst& operator*() { - return _variant; + return variant_; } private: - JsonVariantConst _variant; + JsonVariantConst variant_; }; class JsonArrayConstIterator { friend class JsonArray; public: - JsonArrayConstIterator() : _slot(0) {} + JsonArrayConstIterator() : slot_(0) {} explicit JsonArrayConstIterator(const detail::VariantSlot* slot) - : _slot(slot) {} + : slot_(slot) {} JsonVariantConst operator*() const { - return JsonVariantConst(_slot->data()); + return JsonVariantConst(slot_->data()); } VariantConstPtr operator->() { - return VariantConstPtr(_slot->data()); + return VariantConstPtr(slot_->data()); } bool operator==(const JsonArrayConstIterator& other) const { - return _slot == other._slot; + return slot_ == other.slot_; } bool operator!=(const JsonArrayConstIterator& other) const { - return _slot != other._slot; + return slot_ != other.slot_; } JsonArrayConstIterator& operator++() { - _slot = _slot->next(); + slot_ = slot_->next(); return *this; } JsonArrayConstIterator& operator+=(size_t distance) { - _slot = _slot->next(distance); + slot_ = slot_->next(distance); return *this; } private: - const detail::VariantSlot* _slot; + const detail::VariantSlot* slot_; }; ARDUINOJSON_END_PUBLIC_NAMESPACE diff --git a/src/ArduinoJson/Collection/CollectionData.hpp b/src/ArduinoJson/Collection/CollectionData.hpp index aa807b5d..090c98b9 100644 --- a/src/ArduinoJson/Collection/CollectionData.hpp +++ b/src/ArduinoJson/Collection/CollectionData.hpp @@ -16,8 +16,8 @@ class VariantData; class VariantSlot; class CollectionData { - VariantSlot* _head; - VariantSlot* _tail; + VariantSlot* head_; + VariantSlot* tail_; public: // Must be a POD! @@ -67,7 +67,7 @@ class CollectionData { bool copyFrom(const CollectionData& src, MemoryPool* pool); VariantSlot* head() const { - return _head; + return head_; } void movePointers(ptrdiff_t stringDistance, ptrdiff_t variantDistance); diff --git a/src/ArduinoJson/Collection/CollectionImpl.hpp b/src/ArduinoJson/Collection/CollectionImpl.hpp index c99b524b..134d5bd6 100644 --- a/src/ArduinoJson/Collection/CollectionImpl.hpp +++ b/src/ArduinoJson/Collection/CollectionImpl.hpp @@ -16,13 +16,13 @@ inline VariantSlot* CollectionData::addSlot(MemoryPool* pool) { if (!slot) return 0; - if (_tail) { - ARDUINOJSON_ASSERT(pool->owns(_tail)); // Can't alter a linked array/object - _tail->setNextNotNull(slot); - _tail = slot; + if (tail_) { + ARDUINOJSON_ASSERT(pool->owns(tail_)); // Can't alter a linked array/object + tail_->setNextNotNull(slot); + tail_ = slot; } else { - _head = slot; - _tail = slot; + head_ = slot; + tail_ = slot; } slot->clear(); @@ -45,8 +45,8 @@ inline VariantData* CollectionData::addMember(TAdaptedString key, } inline void CollectionData::clear() { - _head = 0; - _tail = 0; + head_ = 0; + tail_ = 0; } template @@ -57,7 +57,7 @@ inline bool CollectionData::containsKey(const TAdaptedString& key) const { inline bool CollectionData::copyFrom(const CollectionData& src, MemoryPool* pool) { clear(); - for (VariantSlot* s = src._head; s; s = s->next()) { + for (VariantSlot* s = src.head_; s; s = s->next()) { VariantData* var; if (s->key() != 0) { JsonString key(s->key(), @@ -78,7 +78,7 @@ template inline VariantSlot* CollectionData::getSlot(TAdaptedString key) const { if (key.isNull()) return 0; - VariantSlot* slot = _head; + VariantSlot* slot = head_; while (slot) { if (stringEquals(key, adaptString(slot->key()))) break; @@ -88,13 +88,13 @@ inline VariantSlot* CollectionData::getSlot(TAdaptedString key) const { } inline VariantSlot* CollectionData::getSlot(size_t index) const { - if (!_head) + if (!head_) return 0; - return _head->next(index); + return head_->next(index); } inline VariantSlot* CollectionData::getPreviousSlot(VariantSlot* target) const { - VariantSlot* current = _head; + VariantSlot* current = head_; while (current) { VariantSlot* next = current->next(); if (next == target) @@ -132,7 +132,7 @@ inline VariantData* CollectionData::getElement(size_t index) const { inline VariantData* CollectionData::getOrAddElement(size_t index, MemoryPool* pool) { - VariantSlot* slot = _head; + VariantSlot* slot = head_; while (slot && index > 0) { slot = slot->next(); index--; @@ -154,9 +154,9 @@ inline void CollectionData::removeSlot(VariantSlot* slot) { if (prev) prev->setNext(next); else - _head = next; + head_ = next; if (!next) - _tail = prev; + tail_ = prev; } inline void CollectionData::removeElement(size_t index) { @@ -165,7 +165,7 @@ inline void CollectionData::removeElement(size_t index) { inline size_t CollectionData::memoryUsage() const { size_t total = 0; - for (VariantSlot* s = _head; s; s = s->next()) { + for (VariantSlot* s = head_; s; s = s->next()) { total += sizeof(VariantSlot) + s->data()->memoryUsage(); if (s->ownsKey()) total += strlen(s->key()) + 1; @@ -174,7 +174,7 @@ inline size_t CollectionData::memoryUsage() const { } inline size_t CollectionData::size() const { - return slotSize(_head); + return slotSize(head_); } template @@ -188,9 +188,9 @@ inline void movePointer(T*& p, ptrdiff_t offset) { inline void CollectionData::movePointers(ptrdiff_t stringDistance, ptrdiff_t variantDistance) { - movePointer(_head, variantDistance); - movePointer(_tail, variantDistance); - for (VariantSlot* slot = _head; slot; slot = slot->next()) + movePointer(head_, variantDistance); + movePointer(tail_, variantDistance); + for (VariantSlot* slot = head_; slot; slot = slot->next()) slot->movePointers(stringDistance, variantDistance); } diff --git a/src/ArduinoJson/Deserialization/DeserializationError.hpp b/src/ArduinoJson/Deserialization/DeserializationError.hpp index 2e97a024..1bfc3939 100644 --- a/src/ArduinoJson/Deserialization/DeserializationError.hpp +++ b/src/ArduinoJson/Deserialization/DeserializationError.hpp @@ -26,49 +26,49 @@ class DeserializationError { }; DeserializationError() {} - DeserializationError(Code c) : _code(c) {} + DeserializationError(Code c) : code_(c) {} // Compare with DeserializationError friend bool operator==(const DeserializationError& lhs, const DeserializationError& rhs) { - return lhs._code == rhs._code; + return lhs.code_ == rhs.code_; } friend bool operator!=(const DeserializationError& lhs, const DeserializationError& rhs) { - return lhs._code != rhs._code; + return lhs.code_ != rhs.code_; } // Compare with Code friend bool operator==(const DeserializationError& lhs, Code rhs) { - return lhs._code == rhs; + return lhs.code_ == rhs; } friend bool operator==(Code lhs, const DeserializationError& rhs) { - return lhs == rhs._code; + return lhs == rhs.code_; } friend bool operator!=(const DeserializationError& lhs, Code rhs) { - return lhs._code != rhs; + return lhs.code_ != rhs; } friend bool operator!=(Code lhs, const DeserializationError& rhs) { - return lhs != rhs._code; + return lhs != rhs.code_; } // Returns true if there is an error explicit operator bool() const { - return _code != Ok; + return code_ != Ok; } // Returns internal enum, useful for switch statement Code code() const { - return _code; + return code_; } const char* c_str() const { static const char* messages[] = { "Ok", "EmptyInput", "IncompleteInput", "InvalidInput", "NoMemory", "TooDeep"}; - ARDUINOJSON_ASSERT(static_cast(_code) < + ARDUINOJSON_ASSERT(static_cast(code_) < sizeof(messages) / sizeof(messages[0])); - return messages[_code]; + return messages[code_]; } #if ARDUINOJSON_ENABLE_PROGMEM @@ -82,12 +82,12 @@ class DeserializationError { ARDUINOJSON_DEFINE_PROGMEM_ARRAY(const char*, messages, {s0, s1, s2, s3, s4, s5}); return reinterpret_cast( - detail::pgm_read(messages + _code)); + detail::pgm_read(messages + code_)); } #endif private: - Code _code; + Code code_; }; #if ARDUINOJSON_ENABLE_STD_STREAM diff --git a/src/ArduinoJson/Deserialization/Filter.hpp b/src/ArduinoJson/Deserialization/Filter.hpp index 8857cedf..39883027 100644 --- a/src/ArduinoJson/Deserialization/Filter.hpp +++ b/src/ArduinoJson/Deserialization/Filter.hpp @@ -11,34 +11,34 @@ ARDUINOJSON_BEGIN_PUBLIC_NAMESPACE namespace DeserializationOption { class Filter { public: - explicit Filter(JsonVariantConst v) : _variant(v) {} + explicit Filter(JsonVariantConst v) : variant_(v) {} bool allow() const { - return _variant; + return variant_; } bool allowArray() const { - return _variant == true || _variant.is(); + return variant_ == true || variant_.is(); } bool allowObject() const { - return _variant == true || _variant.is(); + return variant_ == true || variant_.is(); } bool allowValue() const { - return _variant == true; + return variant_ == true; } template Filter operator[](const TKey& key) const { - if (_variant == true) // "true" means "allow recursively" + if (variant_ == true) // "true" means "allow recursively" return *this; - JsonVariantConst member = _variant[key]; - return Filter(member.isNull() ? _variant["*"] : member); + JsonVariantConst member = variant_[key]; + return Filter(member.isNull() ? variant_["*"] : member); } private: - JsonVariantConst _variant; + JsonVariantConst variant_; }; } // namespace DeserializationOption diff --git a/src/ArduinoJson/Deserialization/NestingLimit.hpp b/src/ArduinoJson/Deserialization/NestingLimit.hpp index 6d2f6754..6434275b 100644 --- a/src/ArduinoJson/Deserialization/NestingLimit.hpp +++ b/src/ArduinoJson/Deserialization/NestingLimit.hpp @@ -12,20 +12,20 @@ ARDUINOJSON_BEGIN_PUBLIC_NAMESPACE namespace DeserializationOption { class NestingLimit { public: - NestingLimit() : _value(ARDUINOJSON_DEFAULT_NESTING_LIMIT) {} - explicit NestingLimit(uint8_t n) : _value(n) {} + NestingLimit() : value_(ARDUINOJSON_DEFAULT_NESTING_LIMIT) {} + explicit NestingLimit(uint8_t n) : value_(n) {} NestingLimit decrement() const { - ARDUINOJSON_ASSERT(_value > 0); - return NestingLimit(static_cast(_value - 1)); + ARDUINOJSON_ASSERT(value_ > 0); + return NestingLimit(static_cast(value_ - 1)); } bool reached() const { - return _value == 0; + return value_ == 0; } private: - uint8_t _value; + uint8_t value_; }; } // namespace DeserializationOption diff --git a/src/ArduinoJson/Deserialization/Reader.hpp b/src/ArduinoJson/Deserialization/Reader.hpp index e52af6d5..5f18c7b3 100644 --- a/src/ArduinoJson/Deserialization/Reader.hpp +++ b/src/ArduinoJson/Deserialization/Reader.hpp @@ -15,18 +15,18 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE template struct Reader { public: - Reader(TSource& source) : _source(&source) {} + Reader(TSource& source) : source_(&source) {} int read() { - return _source->read(); // Error here? You passed an unsupported input type + return source_->read(); // Error here? You passed an unsupported input type } size_t readBytes(char* buffer, size_t length) { - return _source->readBytes(buffer, length); + return source_->readBytes(buffer, length); } private: - TSource* _source; + TSource* source_; }; template diff --git a/src/ArduinoJson/Deserialization/Readers/ArduinoStreamReader.hpp b/src/ArduinoJson/Deserialization/Readers/ArduinoStreamReader.hpp index ac4f212d..8a873885 100644 --- a/src/ArduinoJson/Deserialization/Readers/ArduinoStreamReader.hpp +++ b/src/ArduinoJson/Deserialization/Readers/ArduinoStreamReader.hpp @@ -12,20 +12,20 @@ template struct Reader::value>::type> { public: - explicit Reader(Stream& stream) : _stream(&stream) {} + explicit Reader(Stream& stream) : stream_(&stream) {} int read() { - // don't use _stream.read() as it ignores the timeout + // don't use stream_.read() as it ignores the timeout char c; - return _stream->readBytes(&c, 1) ? static_cast(c) : -1; + return stream_->readBytes(&c, 1) ? static_cast(c) : -1; } size_t readBytes(char* buffer, size_t length) { - return _stream->readBytes(buffer, length); + return stream_->readBytes(buffer, length); } private: - Stream* _stream; + Stream* stream_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Deserialization/Readers/FlashReader.hpp b/src/ArduinoJson/Deserialization/Readers/FlashReader.hpp index da6880de..df2314ef 100644 --- a/src/ArduinoJson/Deserialization/Readers/FlashReader.hpp +++ b/src/ArduinoJson/Deserialization/Readers/FlashReader.hpp @@ -10,45 +10,45 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE template <> struct Reader { - const char* _ptr; + const char* ptr_; public: explicit Reader(const __FlashStringHelper* ptr) - : _ptr(reinterpret_cast(ptr)) {} + : ptr_(reinterpret_cast(ptr)) {} int read() { - return pgm_read_byte(_ptr++); + return pgm_read_byte(ptr_++); } size_t readBytes(char* buffer, size_t length) { - memcpy_P(buffer, _ptr, length); - _ptr += length; + memcpy_P(buffer, ptr_, length); + ptr_ += length; return length; } }; template <> struct BoundedReader { - const char* _ptr; - const char* _end; + const char* ptr_; + const char* end_; public: explicit BoundedReader(const __FlashStringHelper* ptr, size_t size) - : _ptr(reinterpret_cast(ptr)), _end(_ptr + size) {} + : ptr_(reinterpret_cast(ptr)), end_(ptr_ + size) {} int read() { - if (_ptr < _end) - return pgm_read_byte(_ptr++); + if (ptr_ < end_) + return pgm_read_byte(ptr_++); else return -1; } size_t readBytes(char* buffer, size_t length) { - size_t available = static_cast(_end - _ptr); + size_t available = static_cast(end_ - ptr_); if (available < length) length = available; - memcpy_P(buffer, _ptr, length); - _ptr += length; + memcpy_P(buffer, ptr_, length); + ptr_ += length; return length; } }; diff --git a/src/ArduinoJson/Deserialization/Readers/IteratorReader.hpp b/src/ArduinoJson/Deserialization/Readers/IteratorReader.hpp index 71890798..c0ca4a77 100644 --- a/src/ArduinoJson/Deserialization/Readers/IteratorReader.hpp +++ b/src/ArduinoJson/Deserialization/Readers/IteratorReader.hpp @@ -8,23 +8,23 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE template class IteratorReader { - TIterator _ptr, _end; + TIterator ptr_, end_; public: explicit IteratorReader(TIterator begin, TIterator end) - : _ptr(begin), _end(end) {} + : ptr_(begin), end_(end) {} int read() { - if (_ptr < _end) - return static_cast(*_ptr++); + if (ptr_ < end_) + return static_cast(*ptr_++); else return -1; } size_t readBytes(char* buffer, size_t length) { size_t i = 0; - while (i < length && _ptr < _end) - buffer[i++] = *_ptr++; + while (i < length && ptr_ < end_) + buffer[i++] = *ptr_++; return i; } }; diff --git a/src/ArduinoJson/Deserialization/Readers/RamReader.hpp b/src/ArduinoJson/Deserialization/Readers/RamReader.hpp index 78d01995..eff67bac 100644 --- a/src/ArduinoJson/Deserialization/Readers/RamReader.hpp +++ b/src/ArduinoJson/Deserialization/Readers/RamReader.hpp @@ -21,19 +21,19 @@ struct IsCharOrVoid : IsCharOrVoid {}; template struct Reader::value>::type> { - const char* _ptr; + const char* ptr_; public: explicit Reader(const void* ptr) - : _ptr(ptr ? reinterpret_cast(ptr) : "") {} + : ptr_(ptr ? reinterpret_cast(ptr) : "") {} int read() { - return static_cast(*_ptr++); + return static_cast(*ptr_++); } size_t readBytes(char* buffer, size_t length) { for (size_t i = 0; i < length; i++) - buffer[i] = *_ptr++; + buffer[i] = *ptr_++; return length; } }; diff --git a/src/ArduinoJson/Deserialization/Readers/StdStreamReader.hpp b/src/ArduinoJson/Deserialization/Readers/StdStreamReader.hpp index ecace426..41e0c00a 100644 --- a/src/ArduinoJson/Deserialization/Readers/StdStreamReader.hpp +++ b/src/ArduinoJson/Deserialization/Readers/StdStreamReader.hpp @@ -12,19 +12,19 @@ template struct Reader::value>::type> { public: - explicit Reader(std::istream& stream) : _stream(&stream) {} + explicit Reader(std::istream& stream) : stream_(&stream) {} int read() { - return _stream->get(); + return stream_->get(); } size_t readBytes(char* buffer, size_t length) { - _stream->read(buffer, static_cast(length)); - return static_cast(_stream->gcount()); + stream_->read(buffer, static_cast(length)); + return static_cast(stream_->gcount()); } private: - std::istream* _stream; + std::istream* stream_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Document/BasicJsonDocument.hpp b/src/ArduinoJson/Document/BasicJsonDocument.hpp index cabc11dc..bc206f02 100644 --- a/src/ArduinoJson/Document/BasicJsonDocument.hpp +++ b/src/ArduinoJson/Document/BasicJsonDocument.hpp @@ -14,27 +14,27 @@ template class AllocatorOwner { public: AllocatorOwner() {} - AllocatorOwner(TAllocator a) : _allocator(a) {} + AllocatorOwner(TAllocator a) : allocator_(a) {} void* allocate(size_t size) { - return _allocator.allocate(size); + return allocator_.allocate(size); } void deallocate(void* ptr) { if (ptr) - _allocator.deallocate(ptr); + allocator_.deallocate(ptr); } void* reallocate(void* ptr, size_t new_size) { - return _allocator.reallocate(ptr, new_size); + return allocator_.reallocate(ptr, new_size); } TAllocator& allocator() { - return _allocator; + return allocator_; } private: - TAllocator _allocator; + TAllocator allocator_; }; // A JsonDocument that uses the provided allocator to allocate its memory pool. @@ -106,18 +106,18 @@ class BasicJsonDocument : AllocatorOwner, public JsonDocument { // Reduces the capacity of the memory pool to match the current usage. // https://arduinojson.org/v6/api/basicjsondocument/shrinktofit/ void shrinkToFit() { - ptrdiff_t bytes_reclaimed = _pool.squash(); + ptrdiff_t bytes_reclaimed = pool_.squash(); if (bytes_reclaimed == 0) return; - void* old_ptr = _pool.buffer(); - void* new_ptr = this->reallocate(old_ptr, _pool.capacity()); + void* old_ptr = pool_.buffer(); + void* new_ptr = this->reallocate(old_ptr, pool_.capacity()); ptrdiff_t ptr_offset = static_cast(new_ptr) - static_cast(old_ptr); - _pool.movePointers(ptr_offset); - _data.movePointers(ptr_offset, ptr_offset - bytes_reclaimed); + pool_.movePointers(ptr_offset); + data_.movePointers(ptr_offset, ptr_offset - bytes_reclaimed); } // Reclaims the memory leaked when removing and replacing values. @@ -141,7 +141,7 @@ class BasicJsonDocument : AllocatorOwner, public JsonDocument { void reallocPool(size_t requiredSize) { size_t capa = detail::addPadding(requiredSize); - if (capa == _pool.capacity()) + if (capa == pool_.capacity()) return; freePool(); replacePool(allocPool(detail::addPadding(requiredSize))); @@ -158,10 +158,10 @@ class BasicJsonDocument : AllocatorOwner, public JsonDocument { void moveAssignFrom(BasicJsonDocument& src) { freePool(); - _data = src._data; - _pool = src._pool; - src._data.setNull(); - src._pool = {0, 0}; + data_ = src.data_; + pool_ = src.pool_; + src.data_.setNull(); + src.pool_ = {0, 0}; } }; diff --git a/src/ArduinoJson/Document/JsonDocument.hpp b/src/ArduinoJson/Document/JsonDocument.hpp index 32dea8df..39c1536f 100644 --- a/src/ArduinoJson/Document/JsonDocument.hpp +++ b/src/ArduinoJson/Document/JsonDocument.hpp @@ -40,8 +40,8 @@ class JsonDocument : public detail::VariantOperators { // Empties the document and resets the memory pool // https://arduinojson.org/v6/api/jsondocument/clear/ void clear() { - _pool.clear(); - _data.setNull(); + pool_.clear(); + data_.setNull(); } // Returns true if the root is of the specified type. @@ -67,31 +67,31 @@ class JsonDocument : public detail::VariantOperators { // Returns the number of used bytes in the memory pool. // https://arduinojson.org/v6/api/jsondocument/memoryusage/ size_t memoryUsage() const { - return _pool.size(); + return pool_.size(); } // Returns trues if the memory pool was too small. // https://arduinojson.org/v6/api/jsondocument/overflowed/ bool overflowed() const { - return _pool.overflowed(); + return pool_.overflowed(); } // Returns the depth (nesting level) of the array. // https://arduinojson.org/v6/api/jsondocument/nesting/ size_t nesting() const { - return variantNesting(&_data); + return variantNesting(&data_); } // Returns the capacity of the memory pool. // https://arduinojson.org/v6/api/jsondocument/capacity/ size_t capacity() const { - return _pool.capacity(); + return pool_.capacity(); } // Returns the number of elements in the root array or object. // https://arduinojson.org/v6/api/jsondocument/size/ size_t size() const { - return _data.size(); + return data_.size(); } // Copies the specified document. @@ -161,14 +161,14 @@ class JsonDocument : public detail::VariantOperators { // https://arduinojson.org/v6/api/jsondocument/containskey/ template bool containsKey(TChar* key) const { - return _data.getMember(detail::adaptString(key)) != 0; + return data_.getMember(detail::adaptString(key)) != 0; } // Returns true if the root object contains the specified key. // https://arduinojson.org/v6/api/jsondocument/containskey/ template bool containsKey(const TString& key) const { - return _data.getMember(detail::adaptString(key)) != 0; + return data_.getMember(detail::adaptString(key)) != 0; } // Gets or sets a root object's member. @@ -197,7 +197,7 @@ class JsonDocument : public detail::VariantOperators { FORCE_INLINE typename detail::enable_if::value, JsonVariantConst>::type operator[](const TString& key) const { - return JsonVariantConst(_data.getMember(detail::adaptString(key))); + return JsonVariantConst(data_.getMember(detail::adaptString(key))); } // Gets a root object's member. @@ -206,7 +206,7 @@ class JsonDocument : public detail::VariantOperators { FORCE_INLINE typename detail::enable_if::value, JsonVariantConst>::type operator[](TChar* key) const { - return JsonVariantConst(_data.getMember(detail::adaptString(key))); + return JsonVariantConst(data_.getMember(detail::adaptString(key))); } // Gets or sets a root array's element. @@ -218,14 +218,14 @@ class JsonDocument : public detail::VariantOperators { // Gets a root array's member. // https://arduinojson.org/v6/api/jsondocument/subscript/ FORCE_INLINE JsonVariantConst operator[](size_t index) const { - return JsonVariantConst(_data.getElement(index)); + return JsonVariantConst(data_.getElement(index)); } // Appends a new (null) element to the root array. // Returns a reference to the new element. // https://arduinojson.org/v6/api/jsondocument/add/ FORCE_INLINE JsonVariant add() { - return JsonVariant(&_pool, _data.addElement(&_pool)); + return JsonVariant(&pool_, data_.addElement(&pool_)); } // Appends a value to the root array. @@ -246,7 +246,7 @@ class JsonDocument : public detail::VariantOperators { // ⚠️ Doesn't release the memory associated with the removed element. // https://arduinojson.org/v6/api/jsondocument/remove/ FORCE_INLINE void remove(size_t index) { - _data.remove(index); + data_.remove(index); } // Removes a member of the root object. @@ -255,7 +255,7 @@ class JsonDocument : public detail::VariantOperators { template FORCE_INLINE typename detail::enable_if::value>::type remove(TChar* key) { - _data.remove(detail::adaptString(key)); + data_.remove(detail::adaptString(key)); } // Removes a member of the root object. @@ -265,7 +265,7 @@ class JsonDocument : public detail::VariantOperators { FORCE_INLINE typename detail::enable_if::value>::type remove(const TString& key) { - _data.remove(detail::adaptString(key)); + data_.remove(detail::adaptString(key)); } FORCE_INLINE operator JsonVariant() { @@ -277,44 +277,44 @@ class JsonDocument : public detail::VariantOperators { } protected: - JsonDocument() : _pool(0, 0) {} + JsonDocument() : pool_(0, 0) {} - JsonDocument(detail::MemoryPool pool) : _pool(pool) {} + JsonDocument(detail::MemoryPool pool) : pool_(pool) {} - JsonDocument(char* buf, size_t capa) : _pool(buf, capa) {} + JsonDocument(char* buf, size_t capa) : pool_(buf, capa) {} ~JsonDocument() {} void replacePool(detail::MemoryPool pool) { - _pool = pool; + pool_ = pool; } JsonVariant getVariant() { - return JsonVariant(&_pool, &_data); + return JsonVariant(&pool_, &data_); } JsonVariantConst getVariant() const { - return JsonVariantConst(&_data); + return JsonVariantConst(&data_); } - detail::MemoryPool _pool; - detail::VariantData _data; + detail::MemoryPool pool_; + detail::VariantData data_; protected: detail::MemoryPool* getPool() { - return &_pool; + return &pool_; } detail::VariantData* getData() { - return &_data; + return &data_; } const detail::VariantData* getData() const { - return &_data; + return &data_; } detail::VariantData* getOrCreateData() { - return &_data; + return &data_; } }; diff --git a/src/ArduinoJson/Document/StaticJsonDocument.hpp b/src/ArduinoJson/Document/StaticJsonDocument.hpp index c95f3a7b..07166664 100644 --- a/src/ArduinoJson/Document/StaticJsonDocument.hpp +++ b/src/ArduinoJson/Document/StaticJsonDocument.hpp @@ -11,14 +11,14 @@ ARDUINOJSON_BEGIN_PUBLIC_NAMESPACE // A JsonDocument with a memory pool on the stack. template class StaticJsonDocument : public JsonDocument { - static const size_t _capacity = + static const size_t capacity_ = detail::AddPadding::value>::value; public: - StaticJsonDocument() : JsonDocument(_buffer, _capacity) {} + StaticJsonDocument() : JsonDocument(buffer_, capacity_) {} StaticJsonDocument(const StaticJsonDocument& src) - : JsonDocument(_buffer, _capacity) { + : JsonDocument(buffer_, capacity_) { set(src); } @@ -27,12 +27,12 @@ class StaticJsonDocument : public JsonDocument { const T& src, typename detail::enable_if< detail::is_convertible::value>::type* = 0) - : JsonDocument(_buffer, _capacity) { + : JsonDocument(buffer_, capacity_) { set(src); } // disambiguate - StaticJsonDocument(JsonVariant src) : JsonDocument(_buffer, _capacity) { + StaticJsonDocument(JsonVariant src) : JsonDocument(buffer_, capacity_) { set(src); } @@ -55,7 +55,7 @@ class StaticJsonDocument : public JsonDocument { } private: - char _buffer[_capacity]; + char buffer_[capacity_]; }; ARDUINOJSON_END_PUBLIC_NAMESPACE diff --git a/src/ArduinoJson/Json/JsonDeserializer.hpp b/src/ArduinoJson/Json/JsonDeserializer.hpp index 0696f0ec..a566aea9 100644 --- a/src/ArduinoJson/Json/JsonDeserializer.hpp +++ b/src/ArduinoJson/Json/JsonDeserializer.hpp @@ -23,10 +23,10 @@ class JsonDeserializer { public: JsonDeserializer(MemoryPool* pool, TReader reader, TStringStorage stringStorage) - : _stringStorage(stringStorage), - _foundSomething(false), - _latch(reader), - _pool(pool) {} + : stringStorage_(stringStorage), + foundSomething_(false), + latch_(reader), + pool_(pool) {} template DeserializationError parse(VariantData& variant, TFilter filter, @@ -35,7 +35,7 @@ class JsonDeserializer { err = parseVariant(variant, filter, nestingLimit); - if (!err && _latch.last() != 0 && !variant.isEnclosed()) { + if (!err && latch_.last() != 0 && !variant.isEnclosed()) { // We don't detect trailing characters earlier, so we need to check now return DeserializationError::InvalidInput; } @@ -45,11 +45,11 @@ class JsonDeserializer { private: char current() { - return _latch.current(); + return latch_.current(); } void move() { - _latch.clear(); + latch_.clear(); } bool eat(char charToSkip) { @@ -173,7 +173,7 @@ class JsonDeserializer { for (;;) { if (memberFilter.allow()) { // Allocate slot in array - VariantData* value = array.addElement(_pool); + VariantData* value = array.addElement(pool_); if (!value) return DeserializationError::NoMemory; @@ -269,7 +269,7 @@ class JsonDeserializer { if (!eat(':')) return DeserializationError::InvalidInput; - JsonString key = _stringStorage.str(); + JsonString key = stringStorage_.str(); TFilter memberFilter = filter[key.c_str()]; @@ -278,10 +278,10 @@ class JsonDeserializer { if (!variant) { // Save key in memory pool. // This MUST be done before adding the slot. - key = _stringStorage.save(); + key = stringStorage_.save(); // Allocate slot in object - VariantSlot* slot = object.addSlot(_pool); + VariantSlot* slot = object.addSlot(pool_); if (!slot) return DeserializationError::NoMemory; @@ -377,7 +377,7 @@ class JsonDeserializer { } DeserializationError::Code parseKey() { - _stringStorage.startString(); + stringStorage_.startString(); if (isQuote(current())) { return parseQuotedString(); } else { @@ -388,13 +388,13 @@ class JsonDeserializer { DeserializationError::Code parseStringValue(VariantData& variant) { DeserializationError::Code err; - _stringStorage.startString(); + stringStorage_.startString(); err = parseQuotedString(); if (err) return err; - variant.setString(_stringStorage.save()); + variant.setString(stringStorage_.save()); return DeserializationError::Ok; } @@ -430,9 +430,9 @@ class JsonDeserializer { if (err) return err; if (codepoint.append(codeunit)) - Utf8::encodeCodepoint(codepoint.value(), _stringStorage); + Utf8::encodeCodepoint(codepoint.value(), stringStorage_); #else - _stringStorage.append('\\'); + stringStorage_.append('\\'); #endif continue; } @@ -444,10 +444,10 @@ class JsonDeserializer { move(); } - _stringStorage.append(c); + stringStorage_.append(c); } - if (!_stringStorage.isValid()) + if (!stringStorage_.isValid()) return DeserializationError::NoMemory; return DeserializationError::Ok; @@ -460,14 +460,14 @@ class JsonDeserializer { if (canBeInNonQuotedString(c)) { // no quotes do { move(); - _stringStorage.append(c); + stringStorage_.append(c); c = current(); } while (canBeInNonQuotedString(c)); } else { return DeserializationError::InvalidInput; } - if (!_stringStorage.isValid()) + if (!stringStorage_.isValid()) return DeserializationError::NoMemory; return DeserializationError::Ok; @@ -516,12 +516,12 @@ class JsonDeserializer { char c = current(); while (canBeInNumber(c) && n < 63) { move(); - _buffer[n++] = c; + buffer_[n++] = c; c = current(); } - _buffer[n] = 0; + buffer_[n] = 0; - if (!parseNumber(_buffer, result)) + if (!parseNumber(buffer_, result)) return DeserializationError::InvalidInput; return DeserializationError::Ok; @@ -585,7 +585,7 @@ class JsonDeserializer { switch (current()) { // end of string case '\0': - return _foundSomething ? DeserializationError::IncompleteInput + return foundSomething_ ? DeserializationError::IncompleteInput : DeserializationError::EmptyInput; // spaces @@ -640,7 +640,7 @@ class JsonDeserializer { #endif default: - _foundSomething = true; + foundSomething_ = true; return DeserializationError::Ok; } } @@ -659,11 +659,11 @@ class JsonDeserializer { return DeserializationError::Ok; } - TStringStorage _stringStorage; - bool _foundSomething; - Latch _latch; - MemoryPool* _pool; - char _buffer[64]; // using a member instead of a local variable because it + TStringStorage stringStorage_; + bool foundSomething_; + Latch latch_; + MemoryPool* pool_; + char buffer_[64]; // using a member instead of a local variable because it // ended in the recursive path after compiler inlined the // code }; diff --git a/src/ArduinoJson/Json/JsonSerializer.hpp b/src/ArduinoJson/Json/JsonSerializer.hpp index e5c263a3..781c9c93 100644 --- a/src/ArduinoJson/Json/JsonSerializer.hpp +++ b/src/ArduinoJson/Json/JsonSerializer.hpp @@ -16,7 +16,7 @@ class JsonSerializer : public Visitor { public: static const bool producesText = true; - JsonSerializer(TWriter writer) : _formatter(writer) {} + JsonSerializer(TWriter writer) : formatter_(writer) {} FORCE_INLINE size_t visitArray(const CollectionData& array) { write('['); @@ -43,7 +43,7 @@ class JsonSerializer : public Visitor { const VariantSlot* slot = object.head(); while (slot != 0) { - _formatter.writeString(slot->key()); + formatter_.writeString(slot->key()); write(':'); slot->data()->accept(*this); @@ -59,60 +59,60 @@ class JsonSerializer : public Visitor { } size_t visitFloat(JsonFloat value) { - _formatter.writeFloat(value); + formatter_.writeFloat(value); return bytesWritten(); } size_t visitString(const char* value) { - _formatter.writeString(value); + formatter_.writeString(value); return bytesWritten(); } size_t visitString(const char* value, size_t n) { - _formatter.writeString(value, n); + formatter_.writeString(value, n); return bytesWritten(); } size_t visitRawJson(const char* data, size_t n) { - _formatter.writeRaw(data, n); + formatter_.writeRaw(data, n); return bytesWritten(); } size_t visitSignedInteger(JsonInteger value) { - _formatter.writeInteger(value); + formatter_.writeInteger(value); return bytesWritten(); } size_t visitUnsignedInteger(JsonUInt value) { - _formatter.writeInteger(value); + formatter_.writeInteger(value); return bytesWritten(); } size_t visitBoolean(bool value) { - _formatter.writeBoolean(value); + formatter_.writeBoolean(value); return bytesWritten(); } size_t visitNull() { - _formatter.writeRaw("null"); + formatter_.writeRaw("null"); return bytesWritten(); } protected: size_t bytesWritten() const { - return _formatter.bytesWritten(); + return formatter_.bytesWritten(); } void write(char c) { - _formatter.writeRaw(c); + formatter_.writeRaw(c); } void write(const char* s) { - _formatter.writeRaw(s); + formatter_.writeRaw(s); } private: - TextFormatter _formatter; + TextFormatter formatter_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Json/Latch.hpp b/src/ArduinoJson/Json/Latch.hpp index b36571e8..08070eb3 100644 --- a/src/ArduinoJson/Json/Latch.hpp +++ b/src/ArduinoJson/Json/Latch.hpp @@ -11,45 +11,45 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE template class Latch { public: - Latch(TReader reader) : _reader(reader), _loaded(false) { + Latch(TReader reader) : reader_(reader), loaded_(false) { #if ARDUINOJSON_DEBUG - _ended = false; + ended_ = false; #endif } void clear() { - _loaded = false; + loaded_ = false; } int last() const { - return _current; + return current_; } FORCE_INLINE char current() { - if (!_loaded) { + if (!loaded_) { load(); } - return _current; + return current_; } private: void load() { - ARDUINOJSON_ASSERT(!_ended); - int c = _reader.read(); + ARDUINOJSON_ASSERT(!ended_); + int c = reader_.read(); #if ARDUINOJSON_DEBUG if (c <= 0) - _ended = true; + ended_ = true; #endif - _current = static_cast(c > 0 ? c : 0); - _loaded = true; + current_ = static_cast(c > 0 ? c : 0); + loaded_ = true; } - TReader _reader; - char _current; // NOLINT(clang-analyzer-optin.cplusplus.UninitializedObject) + TReader reader_; + char current_; // NOLINT(clang-analyzer-optin.cplusplus.UninitializedObject) // Not initialized in constructor (+10 bytes on AVR) - bool _loaded; + bool loaded_; #if ARDUINOJSON_DEBUG - bool _ended; + bool ended_; #endif }; diff --git a/src/ArduinoJson/Json/PrettyJsonSerializer.hpp b/src/ArduinoJson/Json/PrettyJsonSerializer.hpp index 974ade37..e8a92064 100644 --- a/src/ArduinoJson/Json/PrettyJsonSerializer.hpp +++ b/src/ArduinoJson/Json/PrettyJsonSerializer.hpp @@ -16,13 +16,13 @@ class PrettyJsonSerializer : public JsonSerializer { typedef JsonSerializer base; public: - PrettyJsonSerializer(TWriter writer) : base(writer), _nesting(0) {} + PrettyJsonSerializer(TWriter writer) : base(writer), nesting_(0) {} size_t visitArray(const CollectionData& array) { const VariantSlot* slot = array.head(); if (slot) { base::write("[\r\n"); - _nesting++; + nesting_++; while (slot != 0) { indent(); slot->data()->accept(*this); @@ -30,7 +30,7 @@ class PrettyJsonSerializer : public JsonSerializer { slot = slot->next(); base::write(slot ? ",\r\n" : "\r\n"); } - _nesting--; + nesting_--; indent(); base::write("]"); } else { @@ -43,7 +43,7 @@ class PrettyJsonSerializer : public JsonSerializer { const VariantSlot* slot = object.head(); if (slot) { base::write("{\r\n"); - _nesting++; + nesting_++; while (slot != 0) { indent(); base::visitString(slot->key()); @@ -53,7 +53,7 @@ class PrettyJsonSerializer : public JsonSerializer { slot = slot->next(); base::write(slot ? ",\r\n" : "\r\n"); } - _nesting--; + nesting_--; indent(); base::write("}"); } else { @@ -64,11 +64,11 @@ class PrettyJsonSerializer : public JsonSerializer { private: void indent() { - for (uint8_t i = 0; i < _nesting; i++) + for (uint8_t i = 0; i < nesting_; i++) base::write(ARDUINOJSON_TAB); } - uint8_t _nesting; + uint8_t nesting_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Json/TextFormatter.hpp b/src/ArduinoJson/Json/TextFormatter.hpp index 6e319aa5..f8eb2822 100644 --- a/src/ArduinoJson/Json/TextFormatter.hpp +++ b/src/ArduinoJson/Json/TextFormatter.hpp @@ -20,13 +20,13 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE template class TextFormatter { public: - explicit TextFormatter(TWriter writer) : _writer(writer) {} + explicit TextFormatter(TWriter writer) : writer_(writer) {} TextFormatter& operator=(const TextFormatter&) = delete; // Returns the number of bytes sent to the TWriter implementation. size_t bytesWritten() const { - return _writer.count(); + return writer_.count(); } void writeBoolean(bool value) { @@ -146,28 +146,28 @@ class TextFormatter { } void writeRaw(const char* s) { - _writer.write(reinterpret_cast(s), strlen(s)); + writer_.write(reinterpret_cast(s), strlen(s)); } void writeRaw(const char* s, size_t n) { - _writer.write(reinterpret_cast(s), n); + writer_.write(reinterpret_cast(s), n); } void writeRaw(const char* begin, const char* end) { - _writer.write(reinterpret_cast(begin), + writer_.write(reinterpret_cast(begin), static_cast(end - begin)); } template void writeRaw(const char (&s)[N]) { - _writer.write(reinterpret_cast(s), N - 1); + writer_.write(reinterpret_cast(s), N - 1); } void writeRaw(char c) { - _writer.write(static_cast(c)); + writer_.write(static_cast(c)); } protected: - CountingDecorator _writer; + CountingDecorator writer_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Json/Utf16.hpp b/src/ArduinoJson/Json/Utf16.hpp index ad22204f..6e314db2 100644 --- a/src/ArduinoJson/Json/Utf16.hpp +++ b/src/ArduinoJson/Json/Utf16.hpp @@ -31,31 +31,31 @@ inline bool isLowSurrogate(uint16_t codeunit) { class Codepoint { public: - Codepoint() : _highSurrogate(0), _codepoint(0) {} + Codepoint() : highSurrogate_(0), codepoint_(0) {} bool append(uint16_t codeunit) { if (isHighSurrogate(codeunit)) { - _highSurrogate = codeunit & 0x3FF; + highSurrogate_ = codeunit & 0x3FF; return false; } if (isLowSurrogate(codeunit)) { - _codepoint = - uint32_t(0x10000 + ((_highSurrogate << 10) | (codeunit & 0x3FF))); + codepoint_ = + uint32_t(0x10000 + ((highSurrogate_ << 10) | (codeunit & 0x3FF))); return true; } - _codepoint = codeunit; + codepoint_ = codeunit; return true; } uint32_t value() const { - return _codepoint; + return codepoint_; } private: - uint16_t _highSurrogate; - uint32_t _codepoint; + uint16_t highSurrogate_; + uint32_t codepoint_; }; } // namespace Utf16 ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Memory/MemoryPool.hpp b/src/ArduinoJson/Memory/MemoryPool.hpp index d1996f1b..754d5851 100644 --- a/src/ArduinoJson/Memory/MemoryPool.hpp +++ b/src/ArduinoJson/Memory/MemoryPool.hpp @@ -26,43 +26,43 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE -// _begin _end +// begin_ end_ // v v // +-------------+--------------+--------------+ // | strings... | (free) | ...variants | // +-------------+--------------+--------------+ // ^ ^ -// _left _right +// left_ right_ class MemoryPool { public: MemoryPool(char* buf, size_t capa) - : _begin(buf), - _left(buf), - _right(buf ? buf + capa : 0), - _end(buf ? buf + capa : 0), - _overflowed(false) { - ARDUINOJSON_ASSERT(isAligned(_begin)); - ARDUINOJSON_ASSERT(isAligned(_right)); - ARDUINOJSON_ASSERT(isAligned(_end)); + : begin_(buf), + left_(buf), + right_(buf ? buf + capa : 0), + end_(buf ? buf + capa : 0), + overflowed_(false) { + ARDUINOJSON_ASSERT(isAligned(begin_)); + ARDUINOJSON_ASSERT(isAligned(right_)); + ARDUINOJSON_ASSERT(isAligned(end_)); } void* buffer() { - return _begin; // NOLINT(clang-analyzer-unix.Malloc) + return begin_; // NOLINT(clang-analyzer-unix.Malloc) // movePointers() alters this pointer } // Gets the capacity of the memoryPool in bytes size_t capacity() const { - return size_t(_end - _begin); + return size_t(end_ - begin_); } size_t size() const { - return size_t(_left - _begin + _end - _right); + return size_t(left_ - begin_ + end_ - right_); } bool overflowed() const { - return _overflowed; + return overflowed_; } VariantSlot* allocVariant() { @@ -91,40 +91,40 @@ class MemoryPool { } void getFreeZone(char** zoneStart, size_t* zoneSize) const { - *zoneStart = _left; - *zoneSize = size_t(_right - _left); + *zoneStart = left_; + *zoneSize = size_t(right_ - left_); } const char* saveStringFromFreeZone(size_t len) { #if ARDUINOJSON_ENABLE_STRING_DEDUPLICATION - const char* dup = findString(adaptString(_left, len)); + const char* dup = findString(adaptString(left_, len)); if (dup) return dup; #endif - const char* str = _left; - _left += len; - *_left++ = 0; + const char* str = left_; + left_ += len; + *left_++ = 0; checkInvariants(); return str; } void markAsOverflowed() { - _overflowed = true; + overflowed_ = true; } void clear() { - _left = _begin; - _right = _end; - _overflowed = false; + left_ = begin_; + right_ = end_; + overflowed_ = false; } bool canAlloc(size_t bytes) const { - return _left + bytes <= _right; + return left_ + bytes <= right_; } bool owns(void* p) const { - return _begin <= p && p < _end; + return begin_ <= p && p < end_; } // Workaround for missing placement new @@ -134,51 +134,51 @@ class MemoryPool { // Squash the free space between strings and variants // - // _begin _end + // begin_ end_ // v v // +-------------+--------------+ // | strings... | ...variants | // +-------------+--------------+ // ^ - // _left _right + // left_ right_ // // This funcion is called before a realloc. ptrdiff_t squash() { - char* new_right = addPadding(_left); - if (new_right >= _right) + char* new_right = addPadding(left_); + if (new_right >= right_) return 0; - size_t right_size = static_cast(_end - _right); - memmove(new_right, _right, right_size); + size_t right_size = static_cast(end_ - right_); + memmove(new_right, right_, right_size); - ptrdiff_t bytes_reclaimed = _right - new_right; - _right = new_right; - _end = new_right + right_size; + ptrdiff_t bytes_reclaimed = right_ - new_right; + right_ = new_right; + end_ = new_right + right_size; return bytes_reclaimed; } // Move all pointers together // This funcion is called after a realloc. void movePointers(ptrdiff_t offset) { - _begin += offset; - _left += offset; - _right += offset; - _end += offset; + begin_ += offset; + left_ += offset; + right_ += offset; + end_ += offset; } private: void checkInvariants() { - ARDUINOJSON_ASSERT(_begin <= _left); - ARDUINOJSON_ASSERT(_left <= _right); - ARDUINOJSON_ASSERT(_right <= _end); - ARDUINOJSON_ASSERT(isAligned(_right)); + ARDUINOJSON_ASSERT(begin_ <= left_); + ARDUINOJSON_ASSERT(left_ <= right_); + ARDUINOJSON_ASSERT(right_ <= end_); + ARDUINOJSON_ASSERT(isAligned(right_)); } #if ARDUINOJSON_ENABLE_STRING_DEDUPLICATION template const char* findString(const TAdaptedString& str) const { size_t n = str.size(); - for (char* next = _begin; next + n < _left; ++next) { + for (char* next = begin_; next + n < left_; ++next) { if (next[n] == '\0' && stringEquals(str, adaptString(next, n))) return next; @@ -192,11 +192,11 @@ class MemoryPool { char* allocString(size_t n) { if (!canAlloc(n)) { - _overflowed = true; + overflowed_ = true; return 0; } - char* s = _left; - _left += n; + char* s = left_; + left_ += n; checkInvariants(); return s; } @@ -208,15 +208,15 @@ class MemoryPool { void* allocRight(size_t bytes) { if (!canAlloc(bytes)) { - _overflowed = true; + overflowed_ = true; return 0; } - _right -= bytes; - return _right; + right_ -= bytes; + return right_; } - char *_begin, *_left, *_right, *_end; - bool _overflowed; + char *begin_, *left_, *right_, *end_; + bool overflowed_; }; template diff --git a/src/ArduinoJson/Misc/SerializedValue.hpp b/src/ArduinoJson/Misc/SerializedValue.hpp index 311f0d25..ae9abd3c 100644 --- a/src/ArduinoJson/Misc/SerializedValue.hpp +++ b/src/ArduinoJson/Misc/SerializedValue.hpp @@ -12,43 +12,43 @@ ARDUINOJSON_BEGIN_PUBLIC_NAMESPACE template class SerializedValue { public: - explicit SerializedValue(T str) : _str(str) {} + explicit SerializedValue(T str) : str_(str) {} operator T() const { - return _str; + return str_; } const char* data() const { - return _str.c_str(); + return str_.c_str(); } size_t size() const { // CAUTION: the old Arduino String doesn't have size() - return _str.length(); + return str_.length(); } private: - T _str; + T str_; }; template class SerializedValue { public: - explicit SerializedValue(TChar* p, size_t n) : _data(p), _size(n) {} + explicit SerializedValue(TChar* p, size_t n) : data_(p), size_(n) {} operator TChar*() const { - return _data; + return data_; } TChar* data() const { - return _data; + return data_; } size_t size() const { - return _size; + return size_; } private: - TChar* _data; - size_t _size; + TChar* data_; + size_t size_; }; template diff --git a/src/ArduinoJson/MsgPack/MsgPackDeserializer.hpp b/src/ArduinoJson/MsgPack/MsgPackDeserializer.hpp index 9fb5a4ac..41934295 100644 --- a/src/ArduinoJson/MsgPack/MsgPackDeserializer.hpp +++ b/src/ArduinoJson/MsgPack/MsgPackDeserializer.hpp @@ -18,17 +18,17 @@ class MsgPackDeserializer { public: MsgPackDeserializer(MemoryPool* pool, TReader reader, TStringStorage stringStorage) - : _pool(pool), - _reader(reader), - _stringStorage(stringStorage), - _foundSomething(false) {} + : pool_(pool), + reader_(reader), + stringStorage_(stringStorage), + foundSomething_(false) {} template DeserializationError parse(VariantData& variant, TFilter filter, DeserializationOption::NestingLimit nestingLimit) { DeserializationError::Code err; err = parseVariant(&variant, filter, nestingLimit); - return _foundSomething ? err : DeserializationError::EmptyInput; + return foundSomething_ ? err : DeserializationError::EmptyInput; } private: @@ -43,7 +43,7 @@ class MsgPackDeserializer { if (err) return err; - _foundSomething = true; + foundSomething_ = true; bool allowValue = filter.allowValue(); @@ -224,7 +224,7 @@ class MsgPackDeserializer { } DeserializationError::Code readByte(uint8_t& value) { - int c = _reader.read(); + int c = reader_.read(); if (c < 0) return DeserializationError::IncompleteInput; value = static_cast(c); @@ -232,7 +232,7 @@ class MsgPackDeserializer { } DeserializationError::Code readBytes(uint8_t* p, size_t n) { - if (_reader.readBytes(reinterpret_cast(p), n) == n) + if (reader_.readBytes(reinterpret_cast(p), n) == n) return DeserializationError::Ok; return DeserializationError::IncompleteInput; } @@ -244,7 +244,7 @@ class MsgPackDeserializer { DeserializationError::Code skipBytes(size_t n) { for (; n; --n) { - if (_reader.read() < 0) + if (reader_.read() < 0) return DeserializationError::IncompleteInput; } return DeserializationError::Ok; @@ -371,14 +371,14 @@ class MsgPackDeserializer { if (err) return err; - variant->setString(_stringStorage.save()); + variant->setString(stringStorage_.save()); return DeserializationError::Ok; } DeserializationError::Code readString(size_t n) { DeserializationError::Code err; - _stringStorage.startString(); + stringStorage_.startString(); for (; n; --n) { uint8_t c; @@ -386,10 +386,10 @@ class MsgPackDeserializer { if (err) return err; - _stringStorage.append(static_cast(c)); + stringStorage_.append(static_cast(c)); } - if (!_stringStorage.isValid()) + if (!stringStorage_.isValid()) return DeserializationError::NoMemory; return DeserializationError::Ok; @@ -435,7 +435,7 @@ class MsgPackDeserializer { if (memberFilter.allow()) { ARDUINOJSON_ASSERT(array != 0); - value = array->addElement(_pool); + value = array->addElement(pool_); if (!value) return DeserializationError::NoMemory; } else { @@ -486,7 +486,7 @@ class MsgPackDeserializer { if (err) return err; - JsonString key = _stringStorage.str(); + JsonString key = stringStorage_.str(); TFilter memberFilter = filter[key.c_str()]; VariantData* member; @@ -495,9 +495,9 @@ class MsgPackDeserializer { // Save key in memory pool. // This MUST be done before adding the slot. - key = _stringStorage.save(); + key = stringStorage_.save(); - VariantSlot* slot = object->addSlot(_pool); + VariantSlot* slot = object->addSlot(pool_); if (!slot) return DeserializationError::NoMemory; @@ -554,10 +554,10 @@ class MsgPackDeserializer { return skipBytes(size + 1U); } - MemoryPool* _pool; - TReader _reader; - TStringStorage _stringStorage; - bool _foundSomething; + MemoryPool* pool_; + TReader reader_; + TStringStorage stringStorage_; + bool foundSomething_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/MsgPack/MsgPackSerializer.hpp b/src/ArduinoJson/MsgPack/MsgPackSerializer.hpp index 45bf5fc8..65769a16 100644 --- a/src/ArduinoJson/MsgPack/MsgPackSerializer.hpp +++ b/src/ArduinoJson/MsgPack/MsgPackSerializer.hpp @@ -19,7 +19,7 @@ class MsgPackSerializer : public Visitor { public: static const bool producesText = false; - MsgPackSerializer(TWriter writer) : _writer(writer) {} + MsgPackSerializer(TWriter writer) : writer_(writer) {} template typename enable_if::type visitFloat(T value32) { @@ -177,15 +177,15 @@ class MsgPackSerializer : public Visitor { private: size_t bytesWritten() const { - return _writer.count(); + return writer_.count(); } void writeByte(uint8_t c) { - _writer.write(c); + writer_.write(c); } void writeBytes(const uint8_t* p, size_t n) { - _writer.write(p, n); + writer_.write(p, n); } template @@ -194,7 +194,7 @@ class MsgPackSerializer : public Visitor { writeBytes(reinterpret_cast(&value), sizeof(value)); } - CountingDecorator _writer; + CountingDecorator writer_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Object/JsonObject.hpp b/src/ArduinoJson/Object/JsonObject.hpp index f2c73b7e..7cdc1c76 100644 --- a/src/ArduinoJson/Object/JsonObject.hpp +++ b/src/ArduinoJson/Object/JsonObject.hpp @@ -20,61 +20,61 @@ class JsonObject : public detail::VariantOperators { typedef JsonObjectIterator iterator; // Creates an unbound reference. - FORCE_INLINE JsonObject() : _data(0), _pool(0) {} + FORCE_INLINE JsonObject() : data_(0), pool_(0) {} // INTERNAL USE ONLY FORCE_INLINE JsonObject(detail::MemoryPool* buf, detail::CollectionData* data) - : _data(data), _pool(buf) {} + : data_(data), pool_(buf) {} operator JsonVariant() const { - void* data = _data; // prevent warning cast-align - return JsonVariant(_pool, reinterpret_cast(data)); + void* data = data_; // prevent warning cast-align + return JsonVariant(pool_, reinterpret_cast(data)); } operator JsonObjectConst() const { - return JsonObjectConst(_data); + return JsonObjectConst(data_); } operator JsonVariantConst() const { - return JsonVariantConst(collectionToVariant(_data)); + return JsonVariantConst(collectionToVariant(data_)); } // Returns true if the reference is unbound. // https://arduinojson.org/v6/api/jsonobject/isnull/ FORCE_INLINE bool isNull() const { - return _data == 0; + return data_ == 0; } // Returns true if the reference is bound. // https://arduinojson.org/v6/api/jsonobject/isnull/ FORCE_INLINE operator bool() const { - return _data != 0; + return data_ != 0; } // Returns the number of bytes occupied by the object. // https://arduinojson.org/v6/api/jsonobject/memoryusage/ FORCE_INLINE size_t memoryUsage() const { - return _data ? _data->memoryUsage() : 0; + return data_ ? data_->memoryUsage() : 0; } // Returns the depth (nesting level) of the object. // https://arduinojson.org/v6/api/jsonobject/nesting/ FORCE_INLINE size_t nesting() const { - return variantNesting(collectionToVariant(_data)); + return variantNesting(collectionToVariant(data_)); } // Returns the number of members in the object. // https://arduinojson.org/v6/api/jsonobject/size/ FORCE_INLINE size_t size() const { - return _data ? _data->size() : 0; + return data_ ? data_->size() : 0; } // Returns an iterator to the first key-value pair of the object. // https://arduinojson.org/v6/api/jsonobject/begin/ FORCE_INLINE iterator begin() const { - if (!_data) + if (!data_) return iterator(); - return iterator(_pool, _data->head()); + return iterator(pool_, data_->head()); } // Returns an iterator following the last key-value pair of the object. @@ -87,22 +87,22 @@ class JsonObject : public detail::VariantOperators { // ⚠️ Doesn't release the memory associated with the removed members. // https://arduinojson.org/v6/api/jsonobject/clear/ void clear() const { - if (!_data) + if (!data_) return; - _data->clear(); + data_->clear(); } // Copies an object. // https://arduinojson.org/v6/api/jsonobject/set/ FORCE_INLINE bool set(JsonObjectConst src) { - if (!_data || !src._data) + if (!data_ || !src.data_) return false; - return _data->copyFrom(*src._data, _pool); + return data_->copyFrom(*src.data_, pool_); } // Compares the content of two objects. FORCE_INLINE bool operator==(JsonObject rhs) const { - return JsonObjectConst(_data) == JsonObjectConst(rhs._data); + return JsonObjectConst(data_) == JsonObjectConst(rhs.data_); } // Gets or sets the member with specified key. @@ -129,9 +129,9 @@ class JsonObject : public detail::VariantOperators { // ⚠️ Doesn't release the memory associated with the removed member. // https://arduinojson.org/v6/api/jsonobject/remove/ FORCE_INLINE void remove(iterator it) const { - if (!_data) + if (!data_) return; - _data->removeSlot(it._slot); + data_->removeSlot(it.slot_); } // Removes the member with the specified key. @@ -194,33 +194,33 @@ class JsonObject : public detail::VariantOperators { private: detail::MemoryPool* getPool() const { - return _pool; + return pool_; } detail::VariantData* getData() const { - return detail::collectionToVariant(_data); + return detail::collectionToVariant(data_); } detail::VariantData* getOrCreateData() const { - return detail::collectionToVariant(_data); + return detail::collectionToVariant(data_); } template inline detail::VariantData* getMember(TAdaptedString key) const { - if (!_data) + if (!data_) return 0; - return _data->getMember(key); + return data_->getMember(key); } template void removeMember(TAdaptedString key) const { - if (!_data) + if (!data_) return; - _data->removeMember(key); + data_->removeMember(key); } - detail::CollectionData* _data; - detail::MemoryPool* _pool; + detail::CollectionData* data_; + detail::MemoryPool* pool_; }; template <> diff --git a/src/ArduinoJson/Object/JsonObjectConst.hpp b/src/ArduinoJson/Object/JsonObjectConst.hpp index ee2af92d..e9b5e483 100644 --- a/src/ArduinoJson/Object/JsonObjectConst.hpp +++ b/src/ArduinoJson/Object/JsonObjectConst.hpp @@ -19,51 +19,51 @@ class JsonObjectConst : public detail::VariantOperators { typedef JsonObjectConstIterator iterator; // Creates an unbound reference. - JsonObjectConst() : _data(0) {} + JsonObjectConst() : data_(0) {} // INTERNAL USE ONLY - JsonObjectConst(const detail::CollectionData* data) : _data(data) {} + JsonObjectConst(const detail::CollectionData* data) : data_(data) {} operator JsonVariantConst() const { - return JsonVariantConst(collectionToVariant(_data)); + return JsonVariantConst(collectionToVariant(data_)); } // Returns true if the reference is unbound. // https://arduinojson.org/v6/api/jsonobjectconst/isnull/ FORCE_INLINE bool isNull() const { - return _data == 0; + return data_ == 0; } // Returns true if the reference is bound. // https://arduinojson.org/v6/api/jsonobjectconst/isnull/ FORCE_INLINE operator bool() const { - return _data != 0; + return data_ != 0; } // Returns the number of bytes occupied by the object. // https://arduinojson.org/v6/api/jsonobjectconst/memoryusage/ FORCE_INLINE size_t memoryUsage() const { - return _data ? _data->memoryUsage() : 0; + return data_ ? data_->memoryUsage() : 0; } // Returns the depth (nesting level) of the object. // https://arduinojson.org/v6/api/jsonobjectconst/nesting/ FORCE_INLINE size_t nesting() const { - return variantNesting(collectionToVariant(_data)); + return variantNesting(collectionToVariant(data_)); } // Returns the number of members in the object. // https://arduinojson.org/v6/api/jsonobjectconst/size/ FORCE_INLINE size_t size() const { - return _data ? _data->size() : 0; + return data_ ? data_->size() : 0; } // Returns an iterator to the first key-value pair of the object. // https://arduinojson.org/v6/api/jsonobjectconst/begin/ FORCE_INLINE iterator begin() const { - if (!_data) + if (!data_) return iterator(); - return iterator(_data->head()); + return iterator(data_->head()); } // Returns an iterator following the last key-value pair of the object. @@ -106,10 +106,10 @@ class JsonObjectConst : public detail::VariantOperators { // Compares objects. FORCE_INLINE bool operator==(JsonObjectConst rhs) const { - if (_data == rhs._data) + if (data_ == rhs.data_) return true; - if (!_data || !rhs._data) + if (!data_ || !rhs.data_) return false; size_t count = 0; @@ -123,17 +123,17 @@ class JsonObjectConst : public detail::VariantOperators { private: const detail::VariantData* getData() const { - return collectionToVariant(_data); + return collectionToVariant(data_); } template const detail::VariantData* getMember(TAdaptedString key) const { - if (!_data) + if (!data_) return 0; - return _data->getMember(key); + return data_->getMember(key); } - const detail::CollectionData* _data; + const detail::CollectionData* data_; }; template <> diff --git a/src/ArduinoJson/Object/JsonObjectIterator.hpp b/src/ArduinoJson/Object/JsonObjectIterator.hpp index defcc109..55c09ed7 100644 --- a/src/ArduinoJson/Object/JsonObjectIterator.hpp +++ b/src/ArduinoJson/Object/JsonObjectIterator.hpp @@ -12,112 +12,112 @@ ARDUINOJSON_BEGIN_PUBLIC_NAMESPACE class JsonPairPtr { public: JsonPairPtr(detail::MemoryPool* pool, detail::VariantSlot* slot) - : _pair(pool, slot) {} + : pair_(pool, slot) {} const JsonPair* operator->() const { - return &_pair; + return &pair_; } const JsonPair& operator*() const { - return _pair; + return pair_; } private: - JsonPair _pair; + JsonPair pair_; }; class JsonObjectIterator { friend class JsonObject; public: - JsonObjectIterator() : _slot(0) {} + JsonObjectIterator() : slot_(0) {} explicit JsonObjectIterator(detail::MemoryPool* pool, detail::VariantSlot* slot) - : _pool(pool), _slot(slot) {} + : pool_(pool), slot_(slot) {} JsonPair operator*() const { - return JsonPair(_pool, _slot); + return JsonPair(pool_, slot_); } JsonPairPtr operator->() { - return JsonPairPtr(_pool, _slot); + return JsonPairPtr(pool_, slot_); } bool operator==(const JsonObjectIterator& other) const { - return _slot == other._slot; + return slot_ == other.slot_; } bool operator!=(const JsonObjectIterator& other) const { - return _slot != other._slot; + return slot_ != other.slot_; } JsonObjectIterator& operator++() { - _slot = _slot->next(); + slot_ = slot_->next(); return *this; } JsonObjectIterator& operator+=(size_t distance) { - _slot = _slot->next(distance); + slot_ = slot_->next(distance); return *this; } private: - detail::MemoryPool* _pool; - detail::VariantSlot* _slot; + detail::MemoryPool* pool_; + detail::VariantSlot* slot_; }; class JsonPairConstPtr { public: - JsonPairConstPtr(const detail::VariantSlot* slot) : _pair(slot) {} + JsonPairConstPtr(const detail::VariantSlot* slot) : pair_(slot) {} const JsonPairConst* operator->() const { - return &_pair; + return &pair_; } const JsonPairConst& operator*() const { - return _pair; + return pair_; } private: - JsonPairConst _pair; + JsonPairConst pair_; }; class JsonObjectConstIterator { friend class JsonObject; public: - JsonObjectConstIterator() : _slot(0) {} + JsonObjectConstIterator() : slot_(0) {} explicit JsonObjectConstIterator(const detail::VariantSlot* slot) - : _slot(slot) {} + : slot_(slot) {} JsonPairConst operator*() const { - return JsonPairConst(_slot); + return JsonPairConst(slot_); } JsonPairConstPtr operator->() { - return JsonPairConstPtr(_slot); + return JsonPairConstPtr(slot_); } bool operator==(const JsonObjectConstIterator& other) const { - return _slot == other._slot; + return slot_ == other.slot_; } bool operator!=(const JsonObjectConstIterator& other) const { - return _slot != other._slot; + return slot_ != other.slot_; } JsonObjectConstIterator& operator++() { - _slot = _slot->next(); + slot_ = slot_->next(); return *this; } JsonObjectConstIterator& operator+=(size_t distance) { - _slot = _slot->next(distance); + slot_ = slot_->next(distance); return *this; } private: - const detail::VariantSlot* _slot; + const detail::VariantSlot* slot_; }; ARDUINOJSON_END_PUBLIC_NAMESPACE diff --git a/src/ArduinoJson/Object/JsonPair.hpp b/src/ArduinoJson/Object/JsonPair.hpp index e1a92022..862a6579 100644 --- a/src/ArduinoJson/Object/JsonPair.hpp +++ b/src/ArduinoJson/Object/JsonPair.hpp @@ -17,25 +17,25 @@ class JsonPair { // INTERNAL USE ONLY JsonPair(detail::MemoryPool* pool, detail::VariantSlot* slot) { if (slot) { - _key = JsonString(slot->key(), slot->ownsKey() ? JsonString::Copied + key_ = JsonString(slot->key(), slot->ownsKey() ? JsonString::Copied : JsonString::Linked); - _value = JsonVariant(pool, slot->data()); + value_ = JsonVariant(pool, slot->data()); } } // Returns the key. JsonString key() const { - return _key; + return key_; } // Returns the value. JsonVariant value() const { - return _value; + return value_; } private: - JsonString _key; - JsonVariant _value; + JsonString key_; + JsonVariant value_; }; // A read-only key-value pair. @@ -44,25 +44,25 @@ class JsonPairConst { public: JsonPairConst(const detail::VariantSlot* slot) { if (slot) { - _key = JsonString(slot->key(), slot->ownsKey() ? JsonString::Copied + key_ = JsonString(slot->key(), slot->ownsKey() ? JsonString::Copied : JsonString::Linked); - _value = JsonVariantConst(slot->data()); + value_ = JsonVariantConst(slot->data()); } } // Returns the key. JsonString key() const { - return _key; + return key_; } // Returns the value. JsonVariantConst value() const { - return _value; + return value_; } private: - JsonString _key; - JsonVariantConst _value; + JsonString key_; + JsonVariantConst value_; }; ARDUINOJSON_END_PUBLIC_NAMESPACE diff --git a/src/ArduinoJson/Object/MemberProxy.hpp b/src/ArduinoJson/Object/MemberProxy.hpp index 6d397db7..78e5a521 100644 --- a/src/ArduinoJson/Object/MemberProxy.hpp +++ b/src/ArduinoJson/Object/MemberProxy.hpp @@ -18,10 +18,10 @@ class MemberProxy public: FORCE_INLINE MemberProxy(TUpstream upstream, TStringRef key) - : _upstream(upstream), _key(key) {} + : upstream_(upstream), key_(key) {} MemberProxy(const MemberProxy& src) - : _upstream(src._upstream), _key(src._key) {} + : upstream_(src.upstream_), key_(src.key_) {} FORCE_INLINE MemberProxy& operator=(const MemberProxy& src) { this->set(src); @@ -42,23 +42,23 @@ class MemberProxy private: FORCE_INLINE MemoryPool* getPool() const { - return VariantAttorney::getPool(_upstream); + return VariantAttorney::getPool(upstream_); } FORCE_INLINE VariantData* getData() const { - return variantGetMember(VariantAttorney::getData(_upstream), - adaptString(_key)); + return variantGetMember(VariantAttorney::getData(upstream_), + adaptString(key_)); } FORCE_INLINE VariantData* getOrCreateData() const { - return variantGetOrAddMember(VariantAttorney::getOrCreateData(_upstream), - adaptString(_key), - VariantAttorney::getPool(_upstream)); + return variantGetOrAddMember(VariantAttorney::getOrCreateData(upstream_), + adaptString(key_), + VariantAttorney::getPool(upstream_)); } private: - TUpstream _upstream; - TStringRef _key; + TUpstream upstream_; + TStringRef key_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Polyfills/pgmspace_generic.hpp b/src/ArduinoJson/Polyfills/pgmspace_generic.hpp index 750bb438..dc47581f 100644 --- a/src/ArduinoJson/Polyfills/pgmspace_generic.hpp +++ b/src/ArduinoJson/Polyfills/pgmspace_generic.hpp @@ -54,14 +54,14 @@ inline T pgm_read(const T* p) { template class pgm_ptr { public: - explicit pgm_ptr(const T* ptr) : _ptr(ptr) {} + explicit pgm_ptr(const T* ptr) : ptr_(ptr) {} T operator[](intptr_t index) const { - return pgm_read(_ptr + index); + return pgm_read(ptr_ + index); } private: - const T* _ptr; + const T* ptr_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Polyfills/type_traits/is_convertible.hpp b/src/ArduinoJson/Polyfills/type_traits/is_convertible.hpp index 22d4a065..7c4810f3 100644 --- a/src/ArduinoJson/Polyfills/type_traits/is_convertible.hpp +++ b/src/ArduinoJson/Polyfills/type_traits/is_convertible.hpp @@ -27,10 +27,10 @@ struct is_convertible { static int probe(To); static char probe(...); - static From& _from; + static From& from_; public: - static const bool value = sizeof(probe(_from)) == sizeof(int); + static const bool value = sizeof(probe(from_)) == sizeof(int); }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Serialization/CountingDecorator.hpp b/src/ArduinoJson/Serialization/CountingDecorator.hpp index 767d5ace..b38a22df 100644 --- a/src/ArduinoJson/Serialization/CountingDecorator.hpp +++ b/src/ArduinoJson/Serialization/CountingDecorator.hpp @@ -11,23 +11,23 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE template class CountingDecorator { public: - explicit CountingDecorator(TWriter& writer) : _writer(writer), _count(0) {} + explicit CountingDecorator(TWriter& writer) : writer_(writer), count_(0) {} void write(uint8_t c) { - _count += _writer.write(c); + count_ += writer_.write(c); } void write(const uint8_t* s, size_t n) { - _count += _writer.write(s, n); + count_ += writer_.write(s, n); } size_t count() const { - return _count; + return count_; } private: - TWriter _writer; - size_t _count; + TWriter writer_; + size_t count_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Serialization/Writer.hpp b/src/ArduinoJson/Serialization/Writer.hpp index 3d18c97b..f6ba225d 100644 --- a/src/ArduinoJson/Serialization/Writer.hpp +++ b/src/ArduinoJson/Serialization/Writer.hpp @@ -12,18 +12,18 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE template class Writer { public: - explicit Writer(TDestination& dest) : _dest(&dest) {} + explicit Writer(TDestination& dest) : dest_(&dest) {} size_t write(uint8_t c) { - return _dest->write(c); + return dest_->write(c); } size_t write(const uint8_t* s, size_t n) { - return _dest->write(s, n); + return dest_->write(s, n); } private: - TDestination* _dest; + TDestination* dest_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Serialization/Writers/ArduinoStringWriter.hpp b/src/ArduinoJson/Serialization/Writers/ArduinoStringWriter.hpp index 89440cc0..c6432487 100644 --- a/src/ArduinoJson/Serialization/Writers/ArduinoStringWriter.hpp +++ b/src/ArduinoJson/Serialization/Writers/ArduinoStringWriter.hpp @@ -13,8 +13,8 @@ class Writer<::String, void> { static const size_t bufferCapacity = ARDUINOJSON_STRING_BUFFER_SIZE; public: - explicit Writer(::String& str) : _destination(&str) { - _size = 0; + explicit Writer(::String& str) : destination_(&str) { + size_ = 0; } ~Writer() { @@ -22,10 +22,10 @@ class Writer<::String, void> { } size_t write(uint8_t c) { - if (_size + 1 >= bufferCapacity) + if (size_ + 1 >= bufferCapacity) if (flush() != 0) return 0; - _buffer[_size++] = static_cast(c); + buffer_[size_++] = static_cast(c); return 1; } @@ -37,17 +37,17 @@ class Writer<::String, void> { } size_t flush() { - ARDUINOJSON_ASSERT(_size < bufferCapacity); - _buffer[_size] = 0; - if (_destination->concat(_buffer)) - _size = 0; - return _size; + ARDUINOJSON_ASSERT(size_ < bufferCapacity); + buffer_[size_] = 0; + if (destination_->concat(buffer_)) + size_ = 0; + return size_; } private: - ::String* _destination; - char _buffer[bufferCapacity]; - size_t _size; + ::String* destination_; + char buffer_[bufferCapacity]; + size_t size_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Serialization/Writers/PrintWriter.hpp b/src/ArduinoJson/Serialization/Writers/PrintWriter.hpp index e4da4221..49f6a6e9 100644 --- a/src/ArduinoJson/Serialization/Writers/PrintWriter.hpp +++ b/src/ArduinoJson/Serialization/Writers/PrintWriter.hpp @@ -13,18 +13,18 @@ class Writer< TDestination, typename enable_if::value>::type> { public: - explicit Writer(::Print& print) : _print(&print) {} + explicit Writer(::Print& print) : print_(&print) {} size_t write(uint8_t c) { - return _print->write(c); + return print_->write(c); } size_t write(const uint8_t* s, size_t n) { - return _print->write(s, n); + return print_->write(s, n); } private: - ::Print* _print; + ::Print* print_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Serialization/Writers/StdStreamWriter.hpp b/src/ArduinoJson/Serialization/Writers/StdStreamWriter.hpp index 36d0c5bc..fdfead73 100644 --- a/src/ArduinoJson/Serialization/Writers/StdStreamWriter.hpp +++ b/src/ArduinoJson/Serialization/Writers/StdStreamWriter.hpp @@ -13,21 +13,21 @@ class Writer< TDestination, typename enable_if::value>::type> { public: - explicit Writer(std::ostream& os) : _os(&os) {} + explicit Writer(std::ostream& os) : os_(&os) {} size_t write(uint8_t c) { - _os->put(static_cast(c)); + os_->put(static_cast(c)); return 1; } size_t write(const uint8_t* s, size_t n) { - _os->write(reinterpret_cast(s), + os_->write(reinterpret_cast(s), static_cast(n)); return n; } private: - std::ostream* _os; + std::ostream* os_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Serialization/Writers/StdStringWriter.hpp b/src/ArduinoJson/Serialization/Writers/StdStringWriter.hpp index eb9dd3f1..1f8afaa8 100644 --- a/src/ArduinoJson/Serialization/Writers/StdStringWriter.hpp +++ b/src/ArduinoJson/Serialization/Writers/StdStringWriter.hpp @@ -24,20 +24,20 @@ template class Writer::value>::type> { public: - Writer(TDestination& str) : _str(&str) {} + Writer(TDestination& str) : str_(&str) {} size_t write(uint8_t c) { - _str->push_back(static_cast(c)); + str_->push_back(static_cast(c)); return 1; } size_t write(const uint8_t* s, size_t n) { - _str->append(reinterpret_cast(s), n); + str_->append(reinterpret_cast(s), n); return n; } private: - TDestination* _str; + TDestination* str_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/StringStorage/StringCopier.hpp b/src/ArduinoJson/StringStorage/StringCopier.hpp index 0834a4f6..bcd34898 100644 --- a/src/ArduinoJson/StringStorage/StringCopier.hpp +++ b/src/ArduinoJson/StringStorage/StringCopier.hpp @@ -10,19 +10,19 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE class StringCopier { public: - StringCopier(MemoryPool* pool) : _pool(pool) {} + StringCopier(MemoryPool* pool) : pool_(pool) {} void startString() { - _pool->getFreeZone(&_ptr, &_capacity); - _size = 0; - if (_capacity == 0) - _pool->markAsOverflowed(); + pool_->getFreeZone(&ptr_, &capacity_); + size_ = 0; + if (capacity_ == 0) + pool_->markAsOverflowed(); } JsonString save() { - ARDUINOJSON_ASSERT(_ptr); - ARDUINOJSON_ASSERT(_size < _capacity); // needs room for the terminator - return JsonString(_pool->saveStringFromFreeZone(_size), _size, + ARDUINOJSON_ASSERT(ptr_); + ARDUINOJSON_ASSERT(size_ < capacity_); // needs room for the terminator + return JsonString(pool_->saveStringFromFreeZone(size_), size_, JsonString::Copied); } @@ -37,36 +37,36 @@ class StringCopier { } void append(char c) { - if (_size + 1 < _capacity) - _ptr[_size++] = c; + if (size_ + 1 < capacity_) + ptr_[size_++] = c; else - _pool->markAsOverflowed(); + pool_->markAsOverflowed(); } bool isValid() const { - return !_pool->overflowed(); + return !pool_->overflowed(); } size_t size() const { - return _size; + return size_; } JsonString str() const { - ARDUINOJSON_ASSERT(_ptr); - ARDUINOJSON_ASSERT(_size < _capacity); - _ptr[_size] = 0; - return JsonString(_ptr, _size, JsonString::Copied); + ARDUINOJSON_ASSERT(ptr_); + ARDUINOJSON_ASSERT(size_ < capacity_); + ptr_[size_] = 0; + return JsonString(ptr_, size_, JsonString::Copied); } private: - MemoryPool* _pool; + MemoryPool* pool_; // These fields aren't initialized by the constructor but startString() // // NOLINTNEXTLINE(clang-analyzer-optin.cplusplus.UninitializedObject) - char* _ptr; + char* ptr_; // NOLINTNEXTLINE(clang-analyzer-optin.cplusplus.UninitializedObject) - size_t _size, _capacity; + size_t size_, capacity_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/StringStorage/StringMover.hpp b/src/ArduinoJson/StringStorage/StringMover.hpp index a70565cb..584b2531 100644 --- a/src/ArduinoJson/StringStorage/StringMover.hpp +++ b/src/ArduinoJson/StringStorage/StringMover.hpp @@ -11,20 +11,20 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE class StringMover { public: - StringMover(char* ptr) : _writePtr(ptr) {} + StringMover(char* ptr) : writePtr_(ptr) {} void startString() { - _startPtr = _writePtr; + startPtr_ = writePtr_; } FORCE_INLINE JsonString save() { JsonString s = str(); - _writePtr++; + writePtr_++; return s; } void append(char c) { - *_writePtr++ = c; + *writePtr_++ = c; } bool isValid() const { @@ -32,17 +32,17 @@ class StringMover { } JsonString str() const { - _writePtr[0] = 0; // terminator - return JsonString(_startPtr, size(), JsonString::Linked); + writePtr_[0] = 0; // terminator + return JsonString(startPtr_, size(), JsonString::Linked); } size_t size() const { - return size_t(_writePtr - _startPtr); + return size_t(writePtr_ - startPtr_); } private: - char* _writePtr; - char* _startPtr; + char* writePtr_; + char* startPtr_; }; ARDUINOJSON_END_PRIVATE_NAMESPACE diff --git a/src/ArduinoJson/Strings/Adapters/FlashString.hpp b/src/ArduinoJson/Strings/Adapters/FlashString.hpp index 96ef60df..eaffe7e3 100644 --- a/src/ArduinoJson/Strings/Adapters/FlashString.hpp +++ b/src/ArduinoJson/Strings/Adapters/FlashString.hpp @@ -16,20 +16,20 @@ class FlashString { static const size_t typeSortKey = 1; FlashString(const __FlashStringHelper* str, size_t sz) - : _str(reinterpret_cast(str)), _size(sz) {} + : str_(reinterpret_cast(str)), size_(sz) {} bool isNull() const { - return !_str; + return !str_; } char operator[](size_t i) const { - ARDUINOJSON_ASSERT(_str != 0); - ARDUINOJSON_ASSERT(i <= _size); - return static_cast(pgm_read_byte(_str + i)); + ARDUINOJSON_ASSERT(str_ != 0); + ARDUINOJSON_ASSERT(i <= size_); + return static_cast(pgm_read_byte(str_ + i)); } size_t size() const { - return _size; + return size_; } friend bool stringEquals(FlashString a, SizedRamString b) { @@ -38,7 +38,7 @@ class FlashString { ARDUINOJSON_ASSERT(!b.isNull()); if (a.size() != b.size()) return false; - return ::memcmp_P(b.data(), a._str, a._size) == 0; + return ::memcmp_P(b.data(), a.str_, a.size_) == 0; } friend int stringCompare(FlashString a, SizedRamString b) { @@ -46,7 +46,7 @@ class FlashString { ARDUINOJSON_ASSERT(!a.isNull()); ARDUINOJSON_ASSERT(!b.isNull()); size_t minsize = a.size() < b.size() ? a.size() : b.size(); - int res = ::memcmp_P(b.data(), a._str, minsize); + int res = ::memcmp_P(b.data(), a.str_, minsize); if (res) return -res; if (a.size() < b.size()) @@ -58,7 +58,7 @@ class FlashString { friend void stringGetChars(FlashString s, char* p, size_t n) { ARDUINOJSON_ASSERT(s.size() <= n); - ::memcpy_P(p, s._str, n); + ::memcpy_P(p, s.str_, n); } StringStoragePolicy::Copy storagePolicy() const { @@ -66,8 +66,8 @@ class FlashString { } private: - const char* _str; - size_t _size; + const char* str_; + size_t size_; }; template <> diff --git a/src/ArduinoJson/Strings/Adapters/JsonString.hpp b/src/ArduinoJson/Strings/Adapters/JsonString.hpp index 926b9f0e..5db7c208 100644 --- a/src/ArduinoJson/Strings/Adapters/JsonString.hpp +++ b/src/ArduinoJson/Strings/Adapters/JsonString.hpp @@ -13,15 +13,15 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE class JsonStringAdapter : public SizedRamString { public: JsonStringAdapter(const JsonString& s) - : SizedRamString(s.c_str(), s.size()), _linked(s.isLinked()) {} + : SizedRamString(s.c_str(), s.size()), linked_(s.isLinked()) {} StringStoragePolicy::LinkOrCopy storagePolicy() const { - StringStoragePolicy::LinkOrCopy policy = {_linked}; + StringStoragePolicy::LinkOrCopy policy = {linked_}; return policy; } private: - bool _linked; + bool linked_; }; template <> diff --git a/src/ArduinoJson/Strings/Adapters/RamString.hpp b/src/ArduinoJson/Strings/Adapters/RamString.hpp index f2f6a3f6..12c67c94 100644 --- a/src/ArduinoJson/Strings/Adapters/RamString.hpp +++ b/src/ArduinoJson/Strings/Adapters/RamString.hpp @@ -21,31 +21,31 @@ class ZeroTerminatedRamString { public: static const size_t typeSortKey = 3; - ZeroTerminatedRamString(const char* str) : _str(str) {} + ZeroTerminatedRamString(const char* str) : str_(str) {} bool isNull() const { - return !_str; + return !str_; } size_t size() const { - return _str ? ::strlen(_str) : 0; + return str_ ? ::strlen(str_) : 0; } char operator[](size_t i) const { - ARDUINOJSON_ASSERT(_str != 0); + ARDUINOJSON_ASSERT(str_ != 0); ARDUINOJSON_ASSERT(i <= size()); - return _str[i]; + return str_[i]; } const char* data() const { - return _str; + return str_; } friend int stringCompare(ZeroTerminatedRamString a, ZeroTerminatedRamString b) { ARDUINOJSON_ASSERT(!a.isNull()); ARDUINOJSON_ASSERT(!b.isNull()); - return ::strcmp(a._str, b._str); + return ::strcmp(a.str_, b.str_); } friend bool stringEquals(ZeroTerminatedRamString a, @@ -58,7 +58,7 @@ class ZeroTerminatedRamString { } protected: - const char* _str; + const char* str_; }; template @@ -101,24 +101,24 @@ class SizedRamString { public: static const size_t typeSortKey = 2; - SizedRamString(const char* str, size_t sz) : _str(str), _size(sz) {} + SizedRamString(const char* str, size_t sz) : str_(str), size_(sz) {} bool isNull() const { - return !_str; + return !str_; } size_t size() const { - return _size; + return size_; } char operator[](size_t i) const { - ARDUINOJSON_ASSERT(_str != 0); + ARDUINOJSON_ASSERT(str_ != 0); ARDUINOJSON_ASSERT(i <= size()); - return _str[i]; + return str_[i]; } const char* data() const { - return _str; + return str_; } StringStoragePolicy::Copy storagePolicy() const { @@ -126,8 +126,8 @@ class SizedRamString { } protected: - const char* _str; - size_t _size; + const char* str_; + size_t size_; }; template diff --git a/src/ArduinoJson/Strings/JsonString.hpp b/src/ArduinoJson/Strings/JsonString.hpp index 0914200e..538dbc8a 100644 --- a/src/ArduinoJson/Strings/JsonString.hpp +++ b/src/ArduinoJson/Strings/JsonString.hpp @@ -16,51 +16,51 @@ class JsonString { public: enum Ownership { Copied, Linked }; - JsonString() : _data(0), _size(0), _ownership(Linked) {} + JsonString() : data_(0), size_(0), ownership_(Linked) {} JsonString(const char* data, Ownership ownership = Linked) - : _data(data), _size(data ? ::strlen(data) : 0), _ownership(ownership) {} + : data_(data), size_(data ? ::strlen(data) : 0), ownership_(ownership) {} JsonString(const char* data, size_t sz, Ownership ownership = Linked) - : _data(data), _size(sz), _ownership(ownership) {} + : data_(data), size_(sz), ownership_(ownership) {} // Returns a pointer to the characters. const char* c_str() const { - return _data; + return data_; } // Returns true if the string is null. bool isNull() const { - return !_data; + return !data_; } // Returns true if the string is stored by address. // Returns false if the string is stored by copy. bool isLinked() const { - return _ownership == Linked; + return ownership_ == Linked; } // Returns length of the string. size_t size() const { - return _size; + return size_; } // Returns true if the string is non-null explicit operator bool() const { - return _data != 0; + return data_ != 0; } // Returns true if strings are equal. friend bool operator==(JsonString lhs, JsonString rhs) { - if (lhs._size != rhs._size) + if (lhs.size_ != rhs.size_) return false; - if (lhs._data == rhs._data) + if (lhs.data_ == rhs.data_) return true; - if (!lhs._data) + if (!lhs.data_) return false; - if (!rhs._data) + if (!rhs.data_) return false; - return memcmp(lhs._data, rhs._data, lhs._size) == 0; + return memcmp(lhs.data_, rhs.data_, lhs.size_) == 0; } // Returns true if strings differs. @@ -76,9 +76,9 @@ class JsonString { #endif private: - const char* _data; - size_t _size; - Ownership _ownership; + const char* data_; + size_t size_; + Ownership ownership_; }; ARDUINOJSON_END_PUBLIC_NAMESPACE diff --git a/src/ArduinoJson/Variant/ConverterImpl.hpp b/src/ArduinoJson/Variant/ConverterImpl.hpp index f5e1d046..e95b33e5 100644 --- a/src/ArduinoJson/Variant/ConverterImpl.hpp +++ b/src/ArduinoJson/Variant/ConverterImpl.hpp @@ -205,43 +205,43 @@ struct Converter : private detail::VariantAttorney { namespace detail { class MemoryPoolPrint : public Print { public: - MemoryPoolPrint(MemoryPool* pool) : _pool(pool), _size(0) { - pool->getFreeZone(&_string, &_capacity); + MemoryPoolPrint(MemoryPool* pool) : pool_(pool), size_(0) { + pool->getFreeZone(&string_, &capacity_); } JsonString str() { - ARDUINOJSON_ASSERT(_size < _capacity); - return JsonString(_pool->saveStringFromFreeZone(_size), _size, + ARDUINOJSON_ASSERT(size_ < capacity_); + return JsonString(pool_->saveStringFromFreeZone(size_), size_, JsonString::Copied); } size_t write(uint8_t c) { - if (_size >= _capacity) + if (size_ >= capacity_) return 0; - _string[_size++] = char(c); + string_[size_++] = char(c); return 1; } size_t write(const uint8_t* buffer, size_t size) { - if (_size + size >= _capacity) { - _size = _capacity; // mark as overflowed + if (size_ + size >= capacity_) { + size_ = capacity_; // mark as overflowed return 0; } - memcpy(&_string[_size], buffer, size); - _size += size; + memcpy(&string_[size_], buffer, size); + size_ += size; return size; } bool overflowed() const { - return _size >= _capacity; + return size_ >= capacity_; } private: - MemoryPool* _pool; - size_t _size; - char* _string; - size_t _capacity; + MemoryPool* pool_; + size_t size_; + char* string_; + size_t capacity_; }; } // namespace detail diff --git a/src/ArduinoJson/Variant/JsonVariant.hpp b/src/ArduinoJson/Variant/JsonVariant.hpp index 690b38df..d1f122e9 100644 --- a/src/ArduinoJson/Variant/JsonVariant.hpp +++ b/src/ArduinoJson/Variant/JsonVariant.hpp @@ -16,27 +16,27 @@ class JsonVariant : public detail::VariantRefBase, public: // Creates an unbound reference. - JsonVariant() : _data(0), _pool(0) {} + JsonVariant() : data_(0), pool_(0) {} // INTERNAL USE ONLY JsonVariant(detail::MemoryPool* pool, detail::VariantData* data) - : _data(data), _pool(pool) {} + : data_(data), pool_(pool) {} private: FORCE_INLINE detail::MemoryPool* getPool() const { - return _pool; + return pool_; } FORCE_INLINE detail::VariantData* getData() const { - return _data; + return data_; } FORCE_INLINE detail::VariantData* getOrCreateData() const { - return _data; + return data_; } - detail::VariantData* _data; - detail::MemoryPool* _pool; + detail::VariantData* data_; + detail::MemoryPool* pool_; }; template <> diff --git a/src/ArduinoJson/Variant/JsonVariantConst.hpp b/src/ArduinoJson/Variant/JsonVariantConst.hpp index 9b1913b5..606df083 100644 --- a/src/ArduinoJson/Variant/JsonVariantConst.hpp +++ b/src/ArduinoJson/Variant/JsonVariantConst.hpp @@ -30,39 +30,39 @@ class JsonVariantConst : public detail::VariantTag, public: // Creates an unbound reference. - JsonVariantConst() : _data(0) {} + JsonVariantConst() : data_(0) {} // INTERNAL USE ONLY - explicit JsonVariantConst(const detail::VariantData* data) : _data(data) {} + explicit JsonVariantConst(const detail::VariantData* data) : data_(data) {} // Returns true if the value is null or the reference is unbound. // https://arduinojson.org/v6/api/jsonvariantconst/isnull/ FORCE_INLINE bool isNull() const { using namespace detail; - return variantIsNull(_data); + return variantIsNull(data_); } // Returns true if the reference is unbound. FORCE_INLINE bool isUnbound() const { - return !_data; + return !data_; } // Returns the number of bytes occupied by the value. // https://arduinojson.org/v6/api/jsonvariantconst/memoryusage/ FORCE_INLINE size_t memoryUsage() const { - return _data ? _data->memoryUsage() : 0; + return data_ ? data_->memoryUsage() : 0; } // Returns the depth (nesting level) of the value. // https://arduinojson.org/v6/api/jsonvariantconst/nesting/ FORCE_INLINE size_t nesting() const { - return variantNesting(_data); + return variantNesting(data_); } // Returns the size of the array or object. // https://arduinojson.org/v6/api/jsonvariantconst/size/ size_t size() const { - return variantSize(_data); + return variantSize(data_); } // Casts the value to the specified type. @@ -93,7 +93,7 @@ class JsonVariantConst : public detail::VariantTag, // Gets array's element at specified index. // https://arduinojson.org/v6/api/jsonvariantconst/subscript/ FORCE_INLINE JsonVariantConst operator[](size_t index) const { - return JsonVariantConst(variantGetElement(_data, index)); + return JsonVariantConst(variantGetElement(data_, index)); } // Gets object's member with specified key. @@ -102,7 +102,7 @@ class JsonVariantConst : public detail::VariantTag, FORCE_INLINE typename detail::enable_if::value, JsonVariantConst>::type operator[](const TString& key) const { - return JsonVariantConst(variantGetMember(_data, detail::adaptString(key))); + return JsonVariantConst(variantGetMember(data_, detail::adaptString(key))); } // Gets object's member with specified key. @@ -111,7 +111,7 @@ class JsonVariantConst : public detail::VariantTag, FORCE_INLINE typename detail::enable_if::value, JsonVariantConst>::type operator[](TChar* key) const { - return JsonVariantConst(variantGetMember(_data, detail::adaptString(key))); + return JsonVariantConst(variantGetMember(data_, detail::adaptString(key))); } // Returns true if tge object contains the specified key. @@ -134,11 +134,11 @@ class JsonVariantConst : public detail::VariantTag, protected: const detail::VariantData* getData() const { - return _data; + return data_; } private: - const detail::VariantData* _data; + const detail::VariantData* data_; }; ARDUINOJSON_END_PUBLIC_NAMESPACE diff --git a/src/ArduinoJson/Variant/SlotFunctions.hpp b/src/ArduinoJson/Variant/SlotFunctions.hpp index 41ae51d0..8fee77f2 100644 --- a/src/ArduinoJson/Variant/SlotFunctions.hpp +++ b/src/ArduinoJson/Variant/SlotFunctions.hpp @@ -10,17 +10,17 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE struct SlotKeySetter { - SlotKeySetter(VariantSlot* instance) : _instance(instance) {} + SlotKeySetter(VariantSlot* instance) : instance_(instance) {} template void operator()(TStoredString s) { if (!s) return; - ARDUINOJSON_ASSERT(_instance != 0); - _instance->setKey(s); + ARDUINOJSON_ASSERT(instance_ != 0); + instance_->setKey(s); } - VariantSlot* _instance; + VariantSlot* instance_; }; template diff --git a/src/ArduinoJson/Variant/VariantCompare.hpp b/src/ArduinoJson/Variant/VariantCompare.hpp index bb60b8a4..42445c6b 100644 --- a/src/ArduinoJson/Variant/VariantCompare.hpp +++ b/src/ArduinoJson/Variant/VariantCompare.hpp @@ -81,12 +81,12 @@ struct Comparer : NullComparer { }; struct ArrayComparer : ComparerBase { - const CollectionData* _rhs; + const CollectionData* rhs_; - explicit ArrayComparer(const CollectionData& rhs) : _rhs(&rhs) {} + explicit ArrayComparer(const CollectionData& rhs) : rhs_(&rhs) {} CompareResult visitArray(const CollectionData& lhs) { - if (JsonArrayConst(&lhs) == JsonArrayConst(_rhs)) + if (JsonArrayConst(&lhs) == JsonArrayConst(rhs_)) return COMPARE_RESULT_EQUAL; else return COMPARE_RESULT_DIFFER; @@ -94,12 +94,12 @@ struct ArrayComparer : ComparerBase { }; struct ObjectComparer : ComparerBase { - const CollectionData* _rhs; + const CollectionData* rhs_; - explicit ObjectComparer(const CollectionData& rhs) : _rhs(&rhs) {} + explicit ObjectComparer(const CollectionData& rhs) : rhs_(&rhs) {} CompareResult visitObject(const CollectionData& lhs) { - if (JsonObjectConst(&lhs) == JsonObjectConst(_rhs)) + if (JsonObjectConst(&lhs) == JsonObjectConst(rhs_)) return COMPARE_RESULT_EQUAL; else return COMPARE_RESULT_DIFFER; @@ -107,15 +107,15 @@ struct ObjectComparer : ComparerBase { }; struct RawComparer : ComparerBase { - const char* _rhsData; - size_t _rhsSize; + const char* rhsData_; + size_t rhsSize_; explicit RawComparer(const char* rhsData, size_t rhsSize) - : _rhsData(rhsData), _rhsSize(rhsSize) {} + : rhsData_(rhsData), rhsSize_(rhsSize) {} CompareResult visitRawJson(const char* lhsData, size_t lhsSize) { - size_t size = _rhsSize < lhsSize ? _rhsSize : lhsSize; - int n = memcmp(lhsData, _rhsData, size); + size_t size = rhsSize_ < lhsSize ? rhsSize_ : lhsSize; + int n = memcmp(lhsData, rhsData_, size); if (n < 0) return COMPARE_RESULT_LESS; else if (n > 0) diff --git a/src/ArduinoJson/Variant/VariantData.hpp b/src/ArduinoJson/Variant/VariantData.hpp index 92d6fc64..7ef218c0 100644 --- a/src/ArduinoJson/Variant/VariantData.hpp +++ b/src/ArduinoJson/Variant/VariantData.hpp @@ -14,47 +14,47 @@ ARDUINOJSON_BEGIN_PRIVATE_NAMESPACE class VariantData { - VariantContent _content; // must be first to allow cast from array to variant - uint8_t _flags; + VariantContent content_; // must be first to allow cast from array to variant + uint8_t flags_; public: - VariantData() : _flags(VALUE_IS_NULL) {} + VariantData() : flags_(VALUE_IS_NULL) {} void operator=(const VariantData& src) { - _content = src._content; - _flags = uint8_t((_flags & OWNED_KEY_BIT) | (src._flags & ~OWNED_KEY_BIT)); + content_ = src.content_; + flags_ = uint8_t((flags_ & OWNED_KEY_BIT) | (src.flags_ & ~OWNED_KEY_BIT)); } template typename TVisitor::result_type accept(TVisitor& visitor) const { switch (type()) { case VALUE_IS_FLOAT: - return visitor.visitFloat(_content.asFloat); + return visitor.visitFloat(content_.asFloat); case VALUE_IS_ARRAY: - return visitor.visitArray(_content.asCollection); + return visitor.visitArray(content_.asCollection); case VALUE_IS_OBJECT: - return visitor.visitObject(_content.asCollection); + return visitor.visitObject(content_.asCollection); case VALUE_IS_LINKED_STRING: case VALUE_IS_OWNED_STRING: - return visitor.visitString(_content.asString.data, - _content.asString.size); + return visitor.visitString(content_.asString.data, + content_.asString.size); case VALUE_IS_OWNED_RAW: case VALUE_IS_LINKED_RAW: - return visitor.visitRawJson(_content.asString.data, - _content.asString.size); + return visitor.visitRawJson(content_.asString.data, + content_.asString.size); case VALUE_IS_SIGNED_INTEGER: - return visitor.visitSignedInteger(_content.asSignedInteger); + return visitor.visitSignedInteger(content_.asSignedInteger); case VALUE_IS_UNSIGNED_INTEGER: - return visitor.visitUnsignedInteger(_content.asUnsignedInteger); + return visitor.visitUnsignedInteger(content_.asUnsignedInteger); case VALUE_IS_BOOLEAN: - return visitor.visitBoolean(_content.asBoolean != 0); + return visitor.visitBoolean(content_.asBoolean != 0); default: return visitor.visitNull(); @@ -72,7 +72,7 @@ class VariantData { bool asBoolean() const; CollectionData* asArray() { - return isArray() ? &_content.asCollection : 0; + return isArray() ? &content_.asCollection : 0; } const CollectionData* asArray() const { @@ -80,11 +80,11 @@ class VariantData { } const CollectionData* asCollection() const { - return isCollection() ? &_content.asCollection : 0; + return isCollection() ? &content_.asCollection : 0; } CollectionData* asObject() { - return isObject() ? &_content.asCollection : 0; + return isObject() ? &content_.asCollection : 0; } const CollectionData* asObject() const { @@ -94,7 +94,7 @@ class VariantData { bool copyFrom(const VariantData& src, MemoryPool* pool); bool isArray() const { - return (_flags & VALUE_IS_ARRAY) != 0; + return (flags_ & VALUE_IS_ARRAY) != 0; } bool isBoolean() const { @@ -102,17 +102,17 @@ class VariantData { } bool isCollection() const { - return (_flags & COLLECTION_MASK) != 0; + return (flags_ & COLLECTION_MASK) != 0; } template bool isInteger() const { switch (type()) { case VALUE_IS_UNSIGNED_INTEGER: - return canConvertNumber(_content.asUnsignedInteger); + return canConvertNumber(content_.asUnsignedInteger); case VALUE_IS_SIGNED_INTEGER: - return canConvertNumber(_content.asSignedInteger); + return canConvertNumber(content_.asSignedInteger); default: return false; @@ -120,7 +120,7 @@ class VariantData { } bool isFloat() const { - return (_flags & NUMBER_BIT) != 0; + return (flags_ & NUMBER_BIT) != 0; } bool isString() const { @@ -128,7 +128,7 @@ class VariantData { } bool isObject() const { - return (_flags & VALUE_IS_OBJECT) != 0; + return (flags_ & VALUE_IS_OBJECT) != 0; } bool isNull() const { @@ -141,30 +141,30 @@ class VariantData { void remove(size_t index) { if (isArray()) - _content.asCollection.removeElement(index); + content_.asCollection.removeElement(index); } template void remove(TAdaptedString key) { if (isObject()) - _content.asCollection.removeMember(key); + content_.asCollection.removeMember(key); } void setBoolean(bool value) { setType(VALUE_IS_BOOLEAN); - _content.asBoolean = value; + content_.asBoolean = value; } void setFloat(JsonFloat value) { setType(VALUE_IS_FLOAT); - _content.asFloat = value; + content_.asFloat = value; } void setLinkedRaw(SerializedValue value) { if (value.data()) { setType(VALUE_IS_LINKED_RAW); - _content.asString.data = value.data(); - _content.asString.size = value.size(); + content_.asString.data = value.data(); + content_.asString.size = value.size(); } else { setType(VALUE_IS_NULL); } @@ -175,8 +175,8 @@ class VariantData { const char* dup = pool->saveString(adaptString(value.data(), value.size())); if (dup) { setType(VALUE_IS_OWNED_RAW); - _content.asString.data = dup; - _content.asString.size = value.size(); + content_.asString.data = dup; + content_.asString.size = value.size(); return true; } else { setType(VALUE_IS_NULL); @@ -187,13 +187,13 @@ class VariantData { template typename enable_if::value>::type setInteger(T value) { setType(VALUE_IS_UNSIGNED_INTEGER); - _content.asUnsignedInteger = static_cast(value); + content_.asUnsignedInteger = static_cast(value); } template typename enable_if::value>::type setInteger(T value) { setType(VALUE_IS_SIGNED_INTEGER); - _content.asSignedInteger = value; + content_.asSignedInteger = value; } void setNull() { @@ -206,20 +206,20 @@ class VariantData { setType(VALUE_IS_LINKED_STRING); else setType(VALUE_IS_OWNED_STRING); - _content.asString.data = s.c_str(); - _content.asString.size = s.size(); + content_.asString.data = s.c_str(); + content_.asString.size = s.size(); } CollectionData& toArray() { setType(VALUE_IS_ARRAY); - _content.asCollection.clear(); - return _content.asCollection; + content_.asCollection.clear(); + return content_.asCollection; } CollectionData& toObject() { setType(VALUE_IS_OBJECT); - _content.asCollection.clear(); - return _content.asCollection; + content_.asCollection.clear(); + return content_.asCollection; } size_t memoryUsage() const { @@ -228,17 +228,17 @@ class VariantData { case VALUE_IS_OWNED_RAW: // We always add a zero at the end: the deduplication function uses it // to detect the beginning of the next string. - return _content.asString.size + 1; + return content_.asString.size + 1; case VALUE_IS_OBJECT: case VALUE_IS_ARRAY: - return _content.asCollection.memoryUsage(); + return content_.asCollection.memoryUsage(); default: return 0; } } size_t size() const { - return isCollection() ? _content.asCollection.size() : 0; + return isCollection() ? content_.asCollection.size() : 0; } VariantData* addElement(MemoryPool* pool) { @@ -246,7 +246,7 @@ class VariantData { toArray(); if (!isArray()) return 0; - return _content.asCollection.addElement(pool); + return content_.asCollection.addElement(pool); } VariantData* getElement(size_t index) const { @@ -259,7 +259,7 @@ class VariantData { toArray(); if (!isArray()) return 0; - return _content.asCollection.getOrAddElement(index, pool); + return content_.asCollection.getOrAddElement(index, pool); } template @@ -274,18 +274,18 @@ class VariantData { toObject(); if (!isObject()) return 0; - return _content.asCollection.getOrAddMember(key, pool); + return content_.asCollection.getOrAddMember(key, pool); } void movePointers(ptrdiff_t stringDistance, ptrdiff_t variantDistance) { - if (_flags & OWNED_VALUE_BIT) - _content.asString.data += stringDistance; - if (_flags & COLLECTION_MASK) - _content.asCollection.movePointers(stringDistance, variantDistance); + if (flags_ & OWNED_VALUE_BIT) + content_.asString.data += stringDistance; + if (flags_ & COLLECTION_MASK) + content_.asCollection.movePointers(stringDistance, variantDistance); } uint8_t type() const { - return _flags & VALUE_MASK; + return flags_ & VALUE_MASK; } template @@ -300,22 +300,22 @@ class VariantData { private: void setType(uint8_t t) { - _flags &= OWNED_KEY_BIT; - _flags |= t; + flags_ &= OWNED_KEY_BIT; + flags_ |= t; } struct VariantStringSetter { - VariantStringSetter(VariantData* instance) : _instance(instance) {} + VariantStringSetter(VariantData* instance) : instance_(instance) {} template void operator()(TStoredString s) { if (s) - _instance->setString(s); + instance_->setString(s); else - _instance->setNull(); + instance_->setNull(); } - VariantData* _instance; + VariantData* instance_; }; }; diff --git a/src/ArduinoJson/Variant/VariantImpl.hpp b/src/ArduinoJson/Variant/VariantImpl.hpp index 4372949a..66a473bb 100644 --- a/src/ArduinoJson/Variant/VariantImpl.hpp +++ b/src/ArduinoJson/Variant/VariantImpl.hpp @@ -19,16 +19,16 @@ template inline T VariantData::asIntegral() const { switch (type()) { case VALUE_IS_BOOLEAN: - return _content.asBoolean; + return content_.asBoolean; case VALUE_IS_UNSIGNED_INTEGER: - return convertNumber(_content.asUnsignedInteger); + return convertNumber(content_.asUnsignedInteger); case VALUE_IS_SIGNED_INTEGER: - return convertNumber(_content.asSignedInteger); + return convertNumber(content_.asSignedInteger); case VALUE_IS_LINKED_STRING: case VALUE_IS_OWNED_STRING: - return parseNumber(_content.asString.data); + return parseNumber(content_.asString.data); case VALUE_IS_FLOAT: - return convertNumber(_content.asFloat); + return convertNumber(content_.asFloat); default: return 0; } @@ -37,12 +37,12 @@ inline T VariantData::asIntegral() const { inline bool VariantData::asBoolean() const { switch (type()) { case VALUE_IS_BOOLEAN: - return _content.asBoolean; + return content_.asBoolean; case VALUE_IS_SIGNED_INTEGER: case VALUE_IS_UNSIGNED_INTEGER: - return _content.asUnsignedInteger != 0; + return content_.asUnsignedInteger != 0; case VALUE_IS_FLOAT: - return _content.asFloat != 0; + return content_.asFloat != 0; case VALUE_IS_NULL: return false; default: @@ -55,16 +55,16 @@ template inline T VariantData::asFloat() const { switch (type()) { case VALUE_IS_BOOLEAN: - return static_cast(_content.asBoolean); + return static_cast(content_.asBoolean); case VALUE_IS_UNSIGNED_INTEGER: - return static_cast(_content.asUnsignedInteger); + return static_cast(content_.asUnsignedInteger); case VALUE_IS_SIGNED_INTEGER: - return static_cast(_content.asSignedInteger); + return static_cast(content_.asSignedInteger); case VALUE_IS_LINKED_STRING: case VALUE_IS_OWNED_STRING: - return parseNumber(_content.asString.data); + return parseNumber(content_.asString.data); case VALUE_IS_FLOAT: - return static_cast(_content.asFloat); + return static_cast(content_.asFloat); default: return 0; } @@ -73,10 +73,10 @@ inline T VariantData::asFloat() const { inline JsonString VariantData::asString() const { switch (type()) { case VALUE_IS_LINKED_STRING: - return JsonString(_content.asString.data, _content.asString.size, + return JsonString(content_.asString.data, content_.asString.size, JsonString::Linked); case VALUE_IS_OWNED_STRING: - return JsonString(_content.asString.data, _content.asString.size, + return JsonString(content_.asString.data, content_.asString.size, JsonString::Copied); default: return JsonString(); @@ -86,20 +86,20 @@ inline JsonString VariantData::asString() const { inline bool VariantData::copyFrom(const VariantData& src, MemoryPool* pool) { switch (src.type()) { case VALUE_IS_ARRAY: - return toArray().copyFrom(src._content.asCollection, pool); + return toArray().copyFrom(src.content_.asCollection, pool); case VALUE_IS_OBJECT: - return toObject().copyFrom(src._content.asCollection, pool); + return toObject().copyFrom(src.content_.asCollection, pool); case VALUE_IS_OWNED_STRING: { JsonString value = src.asString(); return setString(adaptString(value), pool); } case VALUE_IS_OWNED_RAW: return storeOwnedRaw( - serialized(src._content.asString.data, src._content.asString.size), + serialized(src.content_.asString.data, src.content_.asString.size), pool); default: setType(src.type()); - _content = src._content; + content_ = src.content_; return true; } } diff --git a/src/ArduinoJson/Variant/VariantSlot.hpp b/src/ArduinoJson/Variant/VariantSlot.hpp index ab5d763c..f88e35ba 100644 --- a/src/ArduinoJson/Variant/VariantSlot.hpp +++ b/src/ArduinoJson/Variant/VariantSlot.hpp @@ -17,10 +17,10 @@ class VariantSlot { // CAUTION: same layout as VariantData // we cannot use composition because it adds padding // (+20% on ESP8266 for example) - VariantContent _content; - uint8_t _flags; - VariantSlotDiff _next; - const char* _key; + VariantContent content_; + uint8_t flags_; + VariantSlotDiff next_; + const char* key_; public: // Must be a POD! @@ -30,15 +30,15 @@ class VariantSlot { // - no inheritance VariantData* data() { - return reinterpret_cast(&_content); + return reinterpret_cast(&content_); } const VariantData* data() const { - return reinterpret_cast(&_content); + return reinterpret_cast(&content_); } VariantSlot* next() { - return _next ? this + _next : 0; + return next_ ? this + next_ : 0; } const VariantSlot* next() const { @@ -48,9 +48,9 @@ class VariantSlot { VariantSlot* next(size_t distance) { VariantSlot* slot = this; while (distance--) { - if (!slot->_next) + if (!slot->next_) return 0; - slot += slot->_next; + slot += slot->next_; } return slot; } @@ -64,7 +64,7 @@ class VariantSlot { numeric_limits::lowest()); ARDUINOJSON_ASSERT(!slot || slot - this <= numeric_limits::highest()); - _next = VariantSlotDiff(slot ? slot - this : 0); + next_ = VariantSlotDiff(slot ? slot - this : 0); } void setNextNotNull(VariantSlot* slot) { @@ -73,39 +73,39 @@ class VariantSlot { numeric_limits::lowest()); ARDUINOJSON_ASSERT(slot - this <= numeric_limits::highest()); - _next = VariantSlotDiff(slot - this); + next_ = VariantSlotDiff(slot - this); } void setKey(JsonString k) { ARDUINOJSON_ASSERT(k); if (k.isLinked()) - _flags &= VALUE_MASK; + flags_ &= VALUE_MASK; else - _flags |= OWNED_KEY_BIT; - _key = k.c_str(); + flags_ |= OWNED_KEY_BIT; + key_ = k.c_str(); } const char* key() const { - return _key; + return key_; } bool ownsKey() const { - return (_flags & OWNED_KEY_BIT) != 0; + return (flags_ & OWNED_KEY_BIT) != 0; } void clear() { - _next = 0; - _flags = 0; - _key = 0; + next_ = 0; + flags_ = 0; + key_ = 0; } void movePointers(ptrdiff_t stringDistance, ptrdiff_t variantDistance) { - if (_flags & OWNED_KEY_BIT) - _key += stringDistance; - if (_flags & OWNED_VALUE_BIT) - _content.asString.data += stringDistance; - if (_flags & COLLECTION_MASK) - _content.asCollection.movePointers(stringDistance, variantDistance); + if (flags_ & OWNED_KEY_BIT) + key_ += stringDistance; + if (flags_ & OWNED_VALUE_BIT) + content_.asString.data += stringDistance; + if (flags_ & COLLECTION_MASK) + content_.asCollection.movePointers(stringDistance, variantDistance); } };