Contiguous parser: review fixes (wording, macros, diagnostic positions)
- "default parser" instead of "library parser" in comments and tests - the three comments in json.hpp say the same thing - current() replaces the repeated `p != end ? *p : 0` - the backslashes of each macro are aligned - the docs say that JSON_DIAGNOSTIC_POSITIONS turns the fast path off Signed-off-by: Niels Lohmann <mail@nlohmann.me>
This commit is contained in:
6 files changed
+294
-274
No files matched your search
Generated
+143
-136
@@ -23903,8 +23903,8 @@ namespace detail
|
||||
@brief a parser for JSON text in one contiguous block, for parse(), accept(),
|
||||
and sax_parse()
|
||||
|
||||
The parser and the lexer exchange one token at a time, and the lexer keeps its
|
||||
state (the current character, a pending unget, where the token starts) in
|
||||
The default parser and lexer exchange one token at a time, and the lexer keeps
|
||||
its state (the current character, a pending unget, where the token starts) in
|
||||
members, which every character pays for. This parser reads the block through a
|
||||
local pointer instead, and the grammar, the whitespace and the tokens are one
|
||||
loop, after yyjson's reader: the state stays in registers, and the expected
|
||||
@@ -23912,13 +23912,13 @@ next token is tested where it is expected instead of being dispatched on.
|
||||
|
||||
It reports what it reads to a SAX handler: the one parse() uses to build the
|
||||
value, the one accept() uses, or the user's handler of sax_parse(). On a byte
|
||||
the library parser would reject, it stops before reporting the token, and
|
||||
returns false: the events up to there are those the library parser reports.
|
||||
The caller then runs the library parser on the same input to report the error
|
||||
the default parser would reject, it stops before reporting the token, and
|
||||
returns false: the events up to there are those the default parser reports.
|
||||
The caller then runs the default parser on the same input to report the error
|
||||
with its exact message and position (report_contiguous_error(),
|
||||
json_sax_error_forwarder), and parse() builds no value from what was read.
|
||||
|
||||
What is accepted, and the values reported, are those of the library parser
|
||||
What is accepted, and the values reported, are those of the default parser
|
||||
with default options: the same number grammar and conversions, the same UTF-8
|
||||
and escape rules, a UTF-8 byte order mark at the start, and, without
|
||||
JSON_STRICT_NUL_HANDLING, a NUL byte as the end of the input.
|
||||
@@ -23972,28 +23972,28 @@ class contiguous_parser
|
||||
std::vector<std::uint64_t> outer;
|
||||
|
||||
// the current byte, or 0 at the end (no JSON token starts with 0)
|
||||
#define JSON_CONTIGUOUS_CUR() (p != e ? *p : 0)
|
||||
#define JSON_CONTIGUOUS_CUR() current(p, e)
|
||||
// skip whitespace; most tokens have none before them
|
||||
#define JSON_CONTIGUOUS_WS() \
|
||||
do \
|
||||
{ \
|
||||
if (JSON_HEDLEY_UNLIKELY(JSON_CONTIGUOUS_CUR() <= ' ')) \
|
||||
{ \
|
||||
p = skip_whitespace(p, e); \
|
||||
} \
|
||||
} \
|
||||
#define JSON_CONTIGUOUS_WS() \
|
||||
do \
|
||||
{ \
|
||||
if (JSON_HEDLEY_UNLIKELY(JSON_CONTIGUOUS_CUR() <= ' ')) \
|
||||
{ \
|
||||
p = skip_whitespace(p, e); \
|
||||
} \
|
||||
} \
|
||||
while (false)
|
||||
#define JSON_CONTIGUOUS_OPEN(is_object) \
|
||||
do \
|
||||
{ \
|
||||
#define JSON_CONTIGUOUS_OPEN(is_object) \
|
||||
do \
|
||||
{ \
|
||||
if (JSON_HEDLEY_UNLIKELY(depth != 0 && (depth % 64) == 0)) /* NOLINT(readability/braces) */ \
|
||||
{ \
|
||||
outer.push_back(kinds); \
|
||||
kinds = 0; \
|
||||
} \
|
||||
kinds = (kinds << 1u) | static_cast<std::uint64_t>(is_object); \
|
||||
++depth; \
|
||||
} \
|
||||
{ \
|
||||
outer.push_back(kinds); \
|
||||
kinds = 0; \
|
||||
} \
|
||||
kinds = (kinds << 1u) | static_cast<std::uint64_t>(is_object); \
|
||||
++depth; \
|
||||
} \
|
||||
while (false)
|
||||
|
||||
JSON_CONTIGUOUS_WS();
|
||||
@@ -24026,107 +24026,107 @@ class contiguous_parser
|
||||
// a value in an array or an object: expanded once for each, so that
|
||||
// each dispatch has its own branch history (arrays tend to hold values
|
||||
// of one kind) and the code after a scalar knows where it is
|
||||
#define JSON_CONTIGUOUS_VALUE \
|
||||
switch (JSON_CONTIGUOUS_CUR()) \
|
||||
{ \
|
||||
case '"': \
|
||||
if (JSON_HEDLEY_UNLIKELY(!string(p, false))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
break; \
|
||||
case '{': \
|
||||
++p; \
|
||||
if (JSON_HEDLEY_UNLIKELY(!report(sax->start_object(detail::unknown_size())))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
JSON_CONTIGUOUS_OPEN(1u); \
|
||||
goto object_first; \
|
||||
case '[': \
|
||||
++p; \
|
||||
if (JSON_HEDLEY_UNLIKELY(!report(sax->start_array(detail::unknown_size())))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
JSON_CONTIGUOUS_OPEN(0u); \
|
||||
goto array_first; \
|
||||
case '-': \
|
||||
case '0': \
|
||||
case '1': \
|
||||
case '2': \
|
||||
case '3': \
|
||||
case '4': \
|
||||
case '5': \
|
||||
case '6': \
|
||||
case '7': \
|
||||
case '8': \
|
||||
case '9': \
|
||||
if (JSON_HEDLEY_UNLIKELY(!number(p))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
break; \
|
||||
case 't': \
|
||||
if (JSON_HEDLEY_UNLIKELY(!literal(p, "true") || !report(sax->boolean(true)))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
break; \
|
||||
case 'f': \
|
||||
if (JSON_HEDLEY_UNLIKELY(!literal(p, "false") || !report(sax->boolean(false)))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
break; \
|
||||
case 'n': \
|
||||
if (JSON_HEDLEY_UNLIKELY(!literal(p, "null") || !report(sax->null()))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
break; \
|
||||
default: \
|
||||
return false; \
|
||||
#define JSON_CONTIGUOUS_VALUE \
|
||||
switch (JSON_CONTIGUOUS_CUR()) \
|
||||
{ \
|
||||
case '"': \
|
||||
if (JSON_HEDLEY_UNLIKELY(!string(p, false))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
break; \
|
||||
case '{': \
|
||||
++p; \
|
||||
if (JSON_HEDLEY_UNLIKELY(!report(sax->start_object(detail::unknown_size())))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
JSON_CONTIGUOUS_OPEN(1u); \
|
||||
goto object_first; \
|
||||
case '[': \
|
||||
++p; \
|
||||
if (JSON_HEDLEY_UNLIKELY(!report(sax->start_array(detail::unknown_size())))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
JSON_CONTIGUOUS_OPEN(0u); \
|
||||
goto array_first; \
|
||||
case '-': \
|
||||
case '0': \
|
||||
case '1': \
|
||||
case '2': \
|
||||
case '3': \
|
||||
case '4': \
|
||||
case '5': \
|
||||
case '6': \
|
||||
case '7': \
|
||||
case '8': \
|
||||
case '9': \
|
||||
if (JSON_HEDLEY_UNLIKELY(!number(p))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
break; \
|
||||
case 't': \
|
||||
if (JSON_HEDLEY_UNLIKELY(!literal(p, "true") || !report(sax->boolean(true)))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
break; \
|
||||
case 'f': \
|
||||
if (JSON_HEDLEY_UNLIKELY(!literal(p, "false") || !report(sax->boolean(false)))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
break; \
|
||||
case 'n': \
|
||||
if (JSON_HEDLEY_UNLIKELY(!literal(p, "null") || !report(sax->null()))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
break; \
|
||||
default: \
|
||||
return false; \
|
||||
}
|
||||
|
||||
// what may follow a value in an array: a comma and the next value, or the end
|
||||
#define JSON_CONTIGUOUS_AFTER_ELEMENT \
|
||||
JSON_CONTIGUOUS_WS(); \
|
||||
#define JSON_CONTIGUOUS_AFTER_ELEMENT \
|
||||
JSON_CONTIGUOUS_WS(); \
|
||||
if (JSON_HEDLEY_LIKELY(JSON_CONTIGUOUS_CUR() == ',')) /* NOLINT(readability/braces) */ \
|
||||
{ \
|
||||
++p; \
|
||||
JSON_CONTIGUOUS_WS(); \
|
||||
goto array_value; \
|
||||
} \
|
||||
{ \
|
||||
++p; \
|
||||
JSON_CONTIGUOUS_WS(); \
|
||||
goto array_value; \
|
||||
} \
|
||||
if (JSON_HEDLEY_LIKELY(JSON_CONTIGUOUS_CUR() == ']')) /* NOLINT(readability/braces) */ \
|
||||
{ \
|
||||
++p; \
|
||||
if (JSON_HEDLEY_UNLIKELY(!report(sax->end_array()))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
goto close; \
|
||||
} \
|
||||
{ \
|
||||
++p; \
|
||||
if (JSON_HEDLEY_UNLIKELY(!report(sax->end_array()))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
goto close; \
|
||||
} \
|
||||
return false;
|
||||
|
||||
// what may follow a value in an object: a comma and the next member, or the end
|
||||
#define JSON_CONTIGUOUS_AFTER_MEMBER \
|
||||
JSON_CONTIGUOUS_WS(); \
|
||||
#define JSON_CONTIGUOUS_AFTER_MEMBER \
|
||||
JSON_CONTIGUOUS_WS(); \
|
||||
if (JSON_HEDLEY_LIKELY(JSON_CONTIGUOUS_CUR() == ',')) /* NOLINT(readability/braces) */ \
|
||||
{ \
|
||||
++p; \
|
||||
JSON_CONTIGUOUS_WS(); \
|
||||
goto object_key; \
|
||||
} \
|
||||
{ \
|
||||
++p; \
|
||||
JSON_CONTIGUOUS_WS(); \
|
||||
goto object_key; \
|
||||
} \
|
||||
if (JSON_HEDLEY_LIKELY(JSON_CONTIGUOUS_CUR() == '}')) /* NOLINT(readability/braces) */ \
|
||||
{ \
|
||||
++p; \
|
||||
if (JSON_HEDLEY_UNLIKELY(!report(sax->end_object()))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
goto close; \
|
||||
} \
|
||||
{ \
|
||||
++p; \
|
||||
if (JSON_HEDLEY_UNLIKELY(!report(sax->end_object()))) \
|
||||
{ \
|
||||
return false; \
|
||||
} \
|
||||
goto close; \
|
||||
} \
|
||||
return false;
|
||||
|
||||
array_first:
|
||||
@@ -24253,6 +24253,12 @@ done:
|
||||
return KeepValues;
|
||||
}
|
||||
|
||||
/// the byte at @a p, or 0 at the end @a e (no JSON token starts with 0)
|
||||
static JSON_HEDLEY_ALWAYS_INLINE uchar current(const uchar* p, const uchar* e) noexcept
|
||||
{
|
||||
return p != e ? *p : 0;
|
||||
}
|
||||
|
||||
/// the first byte at or after p that is not whitespace, or e
|
||||
static JSON_HEDLEY_NEVER_INLINE const uchar* skip_whitespace(const uchar* p, const uchar* e) noexcept
|
||||
{
|
||||
@@ -24302,7 +24308,7 @@ done:
|
||||
/// a JSON text that is a single scalar
|
||||
bool scalar(const uchar*& p)
|
||||
{
|
||||
switch (p != end ? *p : 0)
|
||||
switch (current(p, end))
|
||||
{
|
||||
case '"':
|
||||
return string(p, false);
|
||||
@@ -24490,7 +24496,7 @@ done:
|
||||
contiguous path (lexer::scan_number_bulk_contiguous())
|
||||
|
||||
Integers that do not fit their type become floats; floats that overflow
|
||||
are an error, which the library parser reports.
|
||||
are an error, which the default parser reports.
|
||||
*/
|
||||
JSON_HEDLEY_ALWAYS_INLINE bool number(const uchar*& pos)
|
||||
{
|
||||
@@ -24639,7 +24645,7 @@ done:
|
||||
}
|
||||
if (JSON_HEDLEY_UNLIKELY(!std::isfinite(value)))
|
||||
{
|
||||
return false; // the library parser reports the overflow
|
||||
return false; // the default parser reports the overflow
|
||||
}
|
||||
return report(sax->number_float(value, number_text(data, data + i)));
|
||||
}
|
||||
@@ -24677,7 +24683,7 @@ struct can_parse_contiguous
|
||||
@brief report the input behind @a adapter, one block of memory, to @a sax
|
||||
|
||||
@return whether the input is valid and was reported completely; if not,
|
||||
nothing was consumed from @a adapter, and the caller has the library
|
||||
nothing was consumed from @a adapter, and the caller has the default
|
||||
parser report the error
|
||||
*/
|
||||
template<typename BasicJsonType, bool KeepValues, typename SAX, typename InputAdapterType>
|
||||
@@ -24686,7 +24692,7 @@ bool parse_contiguous(InputAdapterType& adapter, SAX& sax, std::true_type /*cont
|
||||
const std::size_t size = adapter.bulk_remaining();
|
||||
if (size == 0)
|
||||
{
|
||||
return false; // the library parser reports the empty input
|
||||
return false; // the default parser reports the empty input
|
||||
}
|
||||
const auto* const first = reinterpret_cast<const unsigned char*>(adapter.bulk_data());
|
||||
contiguous_parser<BasicJsonType, SAX, KeepValues> p(first, first + size, &sax);
|
||||
@@ -24697,7 +24703,7 @@ bool parse_contiguous(InputAdapterType& adapter, SAX& sax, std::true_type /*cont
|
||||
@brief a SAX handler that only reports a parse error, as the handler of parse()
|
||||
does: it throws the exception, unless exceptions are not allowed
|
||||
|
||||
After contiguous_parser has rejected an input, the library parser is run again
|
||||
After contiguous_parser has rejected an input, the default parser is run again
|
||||
with this handler to report the error: the error, its message and its position
|
||||
are those of parse(), without building the values read up to it.
|
||||
*/
|
||||
@@ -24734,7 +24740,7 @@ struct sax_ignores_number_text<json_sax_error_reporter<BasicJsonType>> : std::tr
|
||||
rejected, as parse() reports it: throw its exception, or, without
|
||||
exceptions, return true
|
||||
|
||||
@return false if the library parser accepts the input after all (which would
|
||||
@return false if the default parser accepts the input after all (which would
|
||||
be a disagreement of the two parsers): the caller then parses it
|
||||
*/
|
||||
template<typename BasicJsonType, typename InputAdapterType>
|
||||
@@ -24788,7 +24794,7 @@ bool parse_contiguous(InputAdapterType& /*adapter*/, BasicJsonType& /*result*/,
|
||||
|
||||
@return whether @a accepted is the result of accept(); false if the input is
|
||||
not one block of memory, in which case the caller checks it.
|
||||
contiguous_parser accepts exactly what the library parser accepts.
|
||||
contiguous_parser accepts exactly what the default parser accepts.
|
||||
*/
|
||||
template<typename BasicJsonType, typename InputAdapterType>
|
||||
bool accept_contiguous(InputAdapterType& adapter, bool& accepted, std::true_type /*contiguous*/)
|
||||
@@ -24809,9 +24815,9 @@ bool accept_contiguous(InputAdapterType& /*adapter*/, bool& /*accepted*/, std::f
|
||||
handler of the user
|
||||
|
||||
After contiguous_parser has delivered the events of the input to the user's
|
||||
handler and rejected it, the library parser is run again with this handler:
|
||||
handler and rejected it, the default parser is run again with this handler:
|
||||
the user's handler gets the error, its message and its position from the
|
||||
library parser, as before, but no event twice.
|
||||
default parser, as before, but no event twice.
|
||||
*/
|
||||
template<typename BasicJsonType, typename SAX>
|
||||
class json_sax_error_forwarder : public json_sax_acceptor<BasicJsonType>
|
||||
@@ -24828,7 +24834,7 @@ class json_sax_error_forwarder : public json_sax_acceptor<BasicJsonType>
|
||||
return user->parse_error(position, last_token, ex);
|
||||
}
|
||||
|
||||
/// whether the library parser reported an error
|
||||
/// whether the default parser reported an error
|
||||
bool reported() const noexcept
|
||||
{
|
||||
return forwarded;
|
||||
@@ -24847,8 +24853,8 @@ struct sax_ignores_number_text<json_sax_error_forwarder<BasicJsonType, SAX>> : s
|
||||
a user's handler @a sax
|
||||
|
||||
contiguous_parser reports to @a sax what it reads, and stops at the token that
|
||||
the library parser would reject, before reporting it: the events up to an error
|
||||
are the same. The error itself is then reported by the library parser (see
|
||||
the default parser would reject, before reporting it: the events up to an error
|
||||
are the same. The error itself is then reported by the default parser (see
|
||||
json_sax_error_forwarder).
|
||||
|
||||
@return whether @a result is the result of sax_parse(); false if the input is
|
||||
@@ -24869,14 +24875,14 @@ bool sax_parse_contiguous(InputAdapterType& adapter, SAX* sax, bool& result, std
|
||||
}
|
||||
if (p.aborted())
|
||||
{
|
||||
// the handler returned false, as with the library parser
|
||||
// the handler returned false, as with the default parser
|
||||
result = false;
|
||||
return true;
|
||||
}
|
||||
}
|
||||
json_sax_error_forwarder<BasicJsonType, SAX> forwarder(sax);
|
||||
result = parser<BasicJsonType, InputAdapterType>(InputAdapterType(adapter), nullptr, true, false, false).sax_parse(&forwarder, true);
|
||||
// the library parser must reject the input as well (the events after the
|
||||
// the default parser must reject the input as well (the events after the
|
||||
// byte contiguous_parser stopped at cannot be delivered any more)
|
||||
JSON_ASSERT(forwarder.reported());
|
||||
return true;
|
||||
@@ -41973,7 +41979,7 @@ public:
|
||||
{
|
||||
basic_json result;
|
||||
// JSON in one block of memory, with the default options, is read by
|
||||
// the faster contiguous_parser; on an error, the library parser reads
|
||||
// the faster contiguous_parser; on an error, the default parser reads
|
||||
// it again to report it (see contiguous_parser)
|
||||
using contiguous = std::integral_constant<bool, detail::can_parse_contiguous<basic_json, typename std::decay<InputAdapterType>::type>::value>;
|
||||
if (cb == nullptr && !ignore_comments && !ignore_trailing_commas
|
||||
@@ -41991,8 +41997,8 @@ public:
|
||||
const bool ignore_comments,
|
||||
const bool ignore_trailing_commas)
|
||||
{
|
||||
// JSON in one block of memory is checked by the faster
|
||||
// contiguous_parser (see parse_impl())
|
||||
// JSON in one block of memory, with the default options, is checked by
|
||||
// the faster contiguous_parser (see parse_impl())
|
||||
using adapter_t = typename std::decay<InputAdapterType>::type;
|
||||
bool accepted = false;
|
||||
if (!ignore_comments && !ignore_trailing_commas
|
||||
@@ -42016,8 +42022,9 @@ public:
|
||||
{
|
||||
if (format == input_format_t::json)
|
||||
{
|
||||
// JSON in one block of memory is read by the faster
|
||||
// contiguous_parser, with the default options (see parse_impl())
|
||||
// JSON in one block of memory, with the default options, is read by
|
||||
// the faster contiguous_parser; on an error, the default parser
|
||||
// reads it again to report it (see parse_impl())
|
||||
bool result = false;
|
||||
if (strict && !ignore_comments && !ignore_trailing_commas
|
||||
&& detail::sax_parse_contiguous<basic_json>(ia, sax, result, std::integral_constant<bool, detail::can_parse_contiguous<basic_json, InputAdapterType>::value> {}))
|
||||
|
||||
Reference in new issue
Block a user