Skip to content

Commit 3ee80c9

Browse files
committed
VariantImpl: change setRawString() to return bool
1 parent 3f63835 commit 3ee80c9

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

src/ArduinoJson/Variant/ConverterImpl.hpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -173,10 +173,10 @@ inline detail::enable_if_t<detail::IsString<T>::value> convertToJson(
173173
// SerializedValue<const __FlashStringHelper*>
174174
template <typename T>
175175
struct Converter<SerializedValue<T>> : private detail::VariantAttorney {
176-
static void toJson(SerializedValue<T> src, JsonVariant dst) {
176+
static bool toJson(SerializedValue<T> src, JsonVariant dst) {
177177
auto variant = getImpl(dst);
178178
variant.clear();
179-
variant.setRawString(detail::adaptString(src.data(), src.size()));
179+
return variant.setRawString(detail::adaptString(src.data(), src.size()));
180180
}
181181
};
182182

src/ArduinoJson/Variant/VariantImpl.hpp

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -425,12 +425,14 @@ class VariantImpl {
425425
}
426426

427427
template <typename TAdaptedString>
428-
void setRawString(TAdaptedString value) {
428+
bool setRawString(TAdaptedString value) {
429429
if (!data_)
430-
return;
430+
return false;
431431
auto dup = resources_->saveString(value);
432-
if (dup)
433-
data_->setRawString(dup);
432+
if (!dup)
433+
return false;
434+
data_->setRawString(dup);
435+
return true;
434436
}
435437

436438
template <typename TAdaptedString>

0 commit comments

Comments
 (0)