search

bblanchon / ArduinoJson

📟 JSON library for Arduino and embedded C++. Simple and efficient.
https://arduinojson.org
MIT License
6.63k stars 1.1k forks source link

enable recognizing Arduino's `String` when emulating API on 64 bits #2084

Closed d-a-v closed 2 months ago

d-a-v commented 2 months ago

When emulating Arduino API on 64 bits host, size_t and unsigned int are different.

Arduino's String is then not recognized by StringTraits thus preventing compilation with tons of errors.

This proposal adds the missing matching. It changes nothing when compiling on 32 bits archs.

This issue was introduced by c89a2025ce11ff741a75567ce676e8f3b9cdd761.

coveralls commented 2 months ago

Coverage Status

coverage: 99.581%. remained the same when pulling 0f34ee4abc87596ddf272f4eb2a5826321d70d8e on d-a-v:ArduinoStringEmulatedOn64bits into 0741bd9e680b39a01621dd3aba32753b60526c37 on bblanchon:6.x.

bblanchon commented 2 months ago

I manually squashed and merged the PR into the 6.x and 7.x branches. Thank you for this contribution.

bblanchon commented 1 week ago

This feature is available in ArduinoJson 7.1.0