diff options
-rw-r--r-- | src/codec-json/json.h | 16 | ||||
-rw-r--r-- | src/codec-json/json.tmpl.h | 134 | ||||
-rw-r--r-- | tests/codec-json.cpp | 23 |
3 files changed, 118 insertions, 55 deletions
diff --git a/src/codec-json/json.h b/src/codec-json/json.h index 58b64aa..1fe6bb5 100644 --- a/src/codec-json/json.h +++ b/src/codec-json/json.h @@ -20,6 +20,16 @@ public: data(std::size_t ring_size_):buffer_{ring_size_}{} + data(const std::string_view& view__): + buffer_{view__.size()} + { + auto ptr = reinterpret_cast<const uint8_t*>(view__.data()); + if(!ptr){ + return; + } + buffer_.push(*ptr, view__.size()); + } + buffer& get_buffer(){ return buffer_; } @@ -93,6 +103,12 @@ public: error_or<void> decode(data<Schema, encode::Json>& from_decode, data<Schema, ToEncoding>& to_decode){ buffer_view buff_v{from_decode.get_buffer()}; + auto eov = impl::json_decode<Schema, Schema, ToEncoding>::decode(buff_v, to_decode); + if(eov.is_error()){ + return std::move(eov.get_error()); + } + from_decode.get_buffer().read_advance(buff_v.read_offset()); + return void_t {}; } }; diff --git a/src/codec-json/json.tmpl.h b/src/codec-json/json.tmpl.h index b8cd7aa..0f6c59f 100644 --- a/src/codec-json/json.tmpl.h +++ b/src/codec-json/json.tmpl.h @@ -10,7 +10,7 @@ class json_encode { }; template<typename T, size_t N, typename RootSchema, typename FromEncode> -struct json_encode<schema::Primitive<T,N>, RootSchema, FromEncode> { +struct json_encode<schema::Primitive<T,N>, RootSchema, FromEncode> { static error_or<void> encode(const data<schema::Primitive<T,N>, FromEncode>& from, buffer& to) { auto val = from.get(); std::array<uint8_t, 256> data; @@ -38,58 +38,6 @@ struct json_encode<schema::Primitive<T,N>, RootSchema, FromEncode> { return void_t{}; } - - static error_or<void> decode(buffer_view& buff, data<schema::Primitive<T,N>, FromEncode>& to){ - assert((buff.read() >= '0' && buff.read() <= '9') || - buff.read() == '+' || buff.read() == '-'); - - std::size_t offset = 0; - - if(buff.read() == '-'){ - ++offset; - }else if(buff.read() == '+'){ - return make_error<err::not_supported>(); - } - if (offset >= buff.read_composite_length()) { - return make_error<err::buffer_exhausted>(); - } - if(buff.read(offset) >= '1' && buff.read(offset) <= '9'){ - ++offset; - - if(offset >= buff.read_composite_length()){ - return make_error<err::buffer_exhausted>(); - } - - while(1){ - if (buff.read(offset) >= '0' && buff.read(offset) <= '9'){ - ++offset; - - if(offset >= buff.read_composite_length()){ - return make_error<err::buffer_exhausted>(); - } - continue; - } - break; - } - }else if (buff.read(offset) == '0' ){ - ++offset; - }else{ - return make_error<err::buffer_exhausted>(); - } - - { - std::string_view num_view{reinterpret_cast<char*>(&buff.read()), offset}; - typename native_data_type<schema::Primitive<T,N>>::type result; - auto fc_result = std::from_chars(num_view.data(), num_view.data() + num_view.size(), result); - if(fc_result.ec != std::errc{}){ - return make_error<err::invalid_state>(); - } - - to.set(result); - } - - return void_t{}; - } }; template<typename RootSchema, typename FromEncode> @@ -369,6 +317,86 @@ struct json_encode<schema::Union<schema::Member<T,Key>...>, RootSchema, FromEnco template<typename Schema, typename RootSchema, typename ToDecode> struct json_decode; +template<typename T, size_t N, typename RootSchema, typename ToDecode> +struct json_decode<schema::Primitive<T,N>, RootSchema, ToDecode> { + static error_or<void> decode(buffer_view& buff, data<schema::Primitive<T,N>, ToDecode>& to){ + assert((buff.read() >= '0' && buff.read() <= '9') || + buff.read() == '+' || buff.read() == '-'); + + std::size_t offset = 0; + + if(buff.read() == '-'){ + ++offset; + }else if(buff.read() == '+'){ + return make_error<err::not_supported>(); + } + if (offset >= buff.read_composite_length()) { + return make_error<err::buffer_exhausted>(); + } + if(buff.read(offset) >= '1' && buff.read(offset) <= '9'){ + ++offset; + + if(offset >= buff.read_composite_length()){ + return make_error<err::buffer_exhausted>(); + } + while(1){ + if (buff.read(offset) >= '0' && buff.read(offset) <= '9'){ + ++offset; + + if(offset >= buff.read_composite_length()){ + break; + } + continue; + } + break; + } + }else if (buff.read(offset) == '0' ){ + ++offset; + }else{ + return make_error<err::buffer_exhausted>(); + } + + { + std::string_view num_view{reinterpret_cast<char*>(&buff.read()), offset}; + typename native_data_type<schema::Primitive<T,N>>::type result; + auto fc_result = std::from_chars(num_view.data(), num_view.data() + num_view.size(), result); + if(fc_result.ec != std::errc{}){ + return make_error<err::invalid_state>(); + } + + to.set(result); + } + buff.read_advance(offset); + + return void_t{}; + } +}; + +template<typename T, size_t D, typename RootSchema, typename ToDecode> +struct json_decode<schema::Array<T,D>, RootSchema, ToDecode> { + template<size_t Level> + static error_or<void> decode_level(buffer_view& buff, data<schema::Array<T,D>, ToDecode>& to, std::array<std::size_t, D>& index){ + if constexpr (Level == D) { + auto eov = json_decode<T, RootSchema, ToDecode>::decode(buff, to.at(index)); + if(eov.is_error()){ + return eov; + } + } else { + assert(buff.read() == '['); + buff.read_advance(1); + if( buff.read_composite_length() == 0 ){ + return make_error<err::buffer_exhausted>(); + } + + } + return void_t{}; + } + + static error_or<void> decode(buffer_view& buff, data<schema::Array<T,D>, ToDecode>& to){ + std::array<std::size_t, D> index; + return decode_level<0>(buff, to, index); + } +}; } } diff --git a/tests/codec-json.cpp b/tests/codec-json.cpp index 030b1e4..fec82b0 100644 --- a/tests/codec-json.cpp +++ b/tests/codec-json.cpp @@ -292,12 +292,13 @@ SAW_TEST("Struct write"){ SAW_TEST("Int8 read"){ using namespace saw; data<schema::Int8, encode::Native> native_int; - data<schema::Int8, encode::Json> json_int; + data<schema::Int8, encode::Json> json_int{"43"}; codec<schema::Int8, encode::Json> json_codec; error_or<void> eov = json_codec.decode(json_int, native_int); - SAW_EXPECT(eov.is_value(), "Encoding error"); + std::string enc_val = convert_to_string(json_int.get_buffer()); + SAW_EXPECT(eov.is_value(), std::string{"Encoding error: "} + enc_val); /** * Currently doing this manually. Technically I should convert to std::string for tests @@ -305,4 +306,22 @@ SAW_TEST("Int8 read"){ typename native_data_type<schema::Int8>::type dec_val = 43; SAW_EXPECT( dec_val == native_int.get(), std::string{"Value is not being encoded correctly. Encoded: "} + std::to_string(static_cast<int>(native_int.get()))); } + +SAW_TEST("Int64 read"){ + using namespace saw; + data<schema::Int64, encode::Native> native_int; + data<schema::Int64, encode::Json> json_int{"-453"}; + + codec<schema::Int64, encode::Json> json_codec; + + error_or<void> eov = json_codec.decode(json_int, native_int); + std::string enc_val = convert_to_string(json_int.get_buffer()); + SAW_EXPECT(eov.is_value(), std::string{"Encoding error: "} + enc_val); + + /** + * Currently doing this manually. Technically I should convert to std::string for tests + */ + typename native_data_type<schema::Int64>::type dec_val = -453; + SAW_EXPECT( dec_val == native_int.get(), std::string{"Value is not being encoded correctly. Encoded: "} + std::to_string(static_cast<int>(native_int.get()))); +} } |