Change all size types to size_t

Author: mikee47

src/ObjectBase.cpp

@@ -37,7 +37,7 @@ size_t ObjectBase::readFlash(size_t offset, void* buffer, size_t count) const
 	return flashmem_read(buffer, addr, count);
 }
 
-uint32_t ObjectBase::length() const
+size_t ObjectBase::length() const
 {
 	if(isNull()) {
 		return 0;

src/include/FlashString/Object.hpp

@@ -77,7 +77,7 @@ template <class ObjectType, typename ElementType> class Object : public ObjectBa
 	/**
 	 * @brief Get the length of the array in elements
 	 */
-	FSTR_INLINE uint32_t length() const
+	FSTR_INLINE size_t length() const
 	{
 		return ObjectBase::length() / sizeof(ElementType);
 	}

src/include/FlashString/ObjectBase.hpp

@@ -31,13 +31,13 @@ class ObjectBase
 	/**
 	 * @brief Get the length of the object data in bytes
 	 */
-	uint32_t length() const;
+	size_t length() const;
 
 	/**
 	 * @brief Get the object data size in bytes
 	 * @note Always an integer multiple of 4 bytes
 	 */
-	FSTR_INLINE uint32_t size() const
+	FSTR_INLINE size_t size() const
 	{
 		return ALIGNUP(length());
 	}

src/include/FlashString/String.hpp

@@ -155,7 +155,7 @@ class String : public Object<String, char>
 	 * @brief Get the number of bytes used to store the String
 	 * @note Always an integer multiple of 4 bytes
 	 */
-	uint32_t size() const
+	size_t size() const
 	{
 		return ALIGNUP(Object::length() + 1);
 	}

todo.rst

@@ -1,9 +1,6 @@
 TODO List
 =========
 
-Change all sizes to size_t
-   e.g. length()
-
 Missing methods
    Behaviour of String should reflect WString. Review and add any missing methods.
    Note that implementations for some are likely non-trivial since we cannot assume