From a4f3dddb79e875bae74b8d4f3ebd6a94076e8a9e Mon Sep 17 00:00:00 2001 From: m-holger Date: Sun, 5 Feb 2023 18:39:30 +0000 Subject: Change JSON_dictionary and JSON_array to store JSON objects rather than std::shared_ptr Recognise that JSON objects are effectively shared pointers to JSON_value. --- libqpdf/JSON.cc | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) (limited to 'libqpdf/JSON.cc') diff --git a/libqpdf/JSON.cc b/libqpdf/JSON.cc index 76db652b..a5df6b91 100644 --- a/libqpdf/JSON.cc +++ b/libqpdf/JSON.cc @@ -322,9 +322,9 @@ JSON::addArrayElement(JSON const& val) throw std::runtime_error("JSON::addArrayElement called on non-array"); } if (val.m->value.get()) { - arr->elements.push_back(val.m->value); + arr->elements.push_back(val); } else { - arr->elements.push_back(std::make_shared()); + arr->elements.push_back(makeNull()); } return arr->elements.back(); } @@ -504,11 +504,11 @@ JSON::checkSchemaInternal( } if (sch_dict && (!pattern_key.empty())) { - auto pattern_schema = sch_dict->members[pattern_key].get(); + auto pattern_schema = sch_dict->members[pattern_key].m->value.get(); for (auto const& iter: this_dict->members) { std::string const& key = iter.first; checkSchemaInternal( - this_dict->members[key].get(), + this_dict->members[key].m->value.get(), pattern_schema, flags, errors, @@ -519,8 +519,8 @@ JSON::checkSchemaInternal( std::string const& key = iter.first; if (this_dict->members.count(key)) { checkSchemaInternal( - this_dict->members[key].get(), - iter.second.get(), + this_dict->members[key].m->value.get(), + iter.second.m->value.get(), flags, errors, prefix + "." + key); @@ -557,8 +557,8 @@ JSON::checkSchemaInternal( int i = 0; for (auto const& element: this_arr->elements) { checkSchemaInternal( - element.get(), - sch_arr->elements.at(0).get(), + element.m->value.get(), + sch_arr->elements.at(0).m->value.get(), flags, errors, prefix + "." + std::to_string(i)); @@ -568,7 +568,7 @@ JSON::checkSchemaInternal( QTC::TC("libtests", "JSON schema array for single item"); checkSchemaInternal( this_v, - sch_arr->elements.at(0).get(), + sch_arr->elements.at(0).m->value.get(), flags, errors, prefix); @@ -587,8 +587,8 @@ JSON::checkSchemaInternal( size_t i = 0; for (auto const& element: this_arr->elements) { checkSchemaInternal( - element.get(), - sch_arr->elements.at(i).get(), + element.m->value.get(), + sch_arr->elements.at(i).m->value.get(), flags, errors, prefix + "." + std::to_string(i)); -- cgit v1.2.3-70-g09d2 From 7ae1e80fd6626ce07262656a7d822e68004754ae Mon Sep 17 00:00:00 2001 From: m-holger Date: Sun, 5 Feb 2023 18:53:55 +0000 Subject: Change JSON::Members::value to std::unique_ptr --- include/qpdf/JSON.hh | 10 +++++----- libqpdf/JSON.cc | 30 ++++++++++++++---------------- 2 files changed, 19 insertions(+), 21 deletions(-) (limited to 'libqpdf/JSON.cc') diff --git a/include/qpdf/JSON.hh b/include/qpdf/JSON.hh index 8cd4e2fe..b368df99 100644 --- a/include/qpdf/JSON.hh +++ b/include/qpdf/JSON.hh @@ -425,7 +425,7 @@ class JSON std::function fn; }; - JSON(std::shared_ptr); + JSON(std::unique_ptr); static bool checkSchemaInternal( JSON_value* this_v, @@ -443,13 +443,13 @@ class JSON ~Members() = default; private: - Members(std::shared_ptr); + Members(std::unique_ptr); Members(Members const&) = delete; - std::shared_ptr value; + std::unique_ptr value; // start and end are only populated for objects created by parse - qpdf_offset_t start; - qpdf_offset_t end; + qpdf_offset_t start{0}; + qpdf_offset_t end{0}; }; std::shared_ptr m; diff --git a/libqpdf/JSON.cc b/libqpdf/JSON.cc index a5df6b91..cb60eabc 100644 --- a/libqpdf/JSON.cc +++ b/libqpdf/JSON.cc @@ -9,15 +9,13 @@ #include #include -JSON::Members::Members(std::shared_ptr value) : - value(value), - start(0), - end(0) +JSON::Members::Members(std::unique_ptr value) : + value(std::move(value)) { } -JSON::JSON(std::shared_ptr value) : - m(new Members(value)) +JSON::JSON(std::unique_ptr value) : + m(new Members(std::move(value))) { } @@ -278,7 +276,7 @@ JSON::encode_string(std::string const& str) JSON JSON::makeDictionary() { - return JSON(std::make_shared()); + return JSON(std::make_unique()); } JSON @@ -286,7 +284,7 @@ JSON::addDictionaryMember(std::string const& key, JSON const& val) { if (auto* obj = dynamic_cast(this->m->value.get())) { return obj->members[encode_string(key)] = - val.m->value ? val.m->value : std::make_shared(); + val.m->value ? val : makeNull(); } else { throw std::runtime_error( "JSON::addDictionaryMember called on non-dictionary"); @@ -311,7 +309,7 @@ JSON::checkDictionaryKeySeen(std::string const& key) JSON JSON::makeArray() { - return JSON(std::make_shared()); + return JSON(std::make_unique()); } JSON @@ -332,43 +330,43 @@ JSON::addArrayElement(JSON const& val) JSON JSON::makeString(std::string const& utf8) { - return JSON(std::make_shared(utf8)); + return JSON(std::make_unique(utf8)); } JSON JSON::makeInt(long long int value) { - return JSON(std::make_shared(value)); + return JSON(std::make_unique(value)); } JSON JSON::makeReal(double value) { - return JSON(std::make_shared(value)); + return JSON(std::make_unique(value)); } JSON JSON::makeNumber(std::string const& encoded) { - return JSON(std::make_shared(encoded)); + return JSON(std::make_unique(encoded)); } JSON JSON::makeBool(bool value) { - return JSON(std::make_shared(value)); + return JSON(std::make_unique(value)); } JSON JSON::makeNull() { - return JSON(std::make_shared()); + return JSON(std::make_unique()); } JSON JSON::makeBlob(std::function fn) { - return JSON(std::make_shared(fn)); + return JSON(std::make_unique(fn)); } bool -- cgit v1.2.3-70-g09d2