feedc0de/ArduinoJson
1
0
Fork 0
forked from bblanchon/ArduinoJson
Code Pull Requests Activity

Fix 'std::string_view' has not been declared (issue #1967)

Browse Source 
Ported from 49e2a8d421
This commit is contained in:
Benoit Blanchon
2023-09-13 18:12:18 +02:00
parent b20653ee1a
commit 92c9faf69f
2 changed files with 15 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
extras/tests/Cpp17/string_view.cpp
View File

@ -1,8 +1,16 @@
// ArduinoJson - https://arduinojson.org
// Copyright © 2014-2023, Benoit BLANCHON
// MIT License
// we expect ArduinoJson.h to include <string_view>
// but we don't want it to included accidentally
#undef ARDUINO
#define ARDUINOJSON_ENABLE_STD_STREAM 0
#define ARDUINOJSON_ENABLE_STD_STRING 0
#include <ArduinoJson.h> #include <ArduinoJson.h>
#include <catch.hpp> #include <catch.hpp>
#include <string_view>
#include "Allocators.hpp" #include "Allocators.hpp"
#if !ARDUINOJSON_ENABLE_STRING_VIEW #if !ARDUINOJSON_ENABLE_STRING_VIEW
@ -24,7 +32,7 @@ TEST_CASE("string_view") {
SECTION("JsonDocument::set()") { SECTION("JsonDocument::set()") {
doc.set(std::string_view("123", 2)); doc.set(std::string_view("123", 2));
REQUIRE(doc.as<std::string>() == "12"); REQUIRE(doc.as<std::string_view>() == "12");
} }
SECTION("JsonDocument::operator[]() const") { SECTION("JsonDocument::operator[]() const") {

4
src/ArduinoJson/Variant/ConverterImpl.hpp
View File

@ -13,6 +13,10 @@
# include <string> # include <string>
#endif #endif
#if ARDUINOJSON_ENABLE_STRING_VIEW
# include <string_view>
#endif
ARDUINOJSON_BEGIN_PUBLIC_NAMESPACE ARDUINOJSON_BEGIN_PUBLIC_NAMESPACE
template <typename T, typename Enable> template <typename T, typename Enable>