Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Move to RFC7159 (closes #90) #101

Merged
merged 15 commits into from
Aug 11, 2014
Merged

Move to RFC7159 (closes #90) #101

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
8d53689
(Pretty)Writer: drop restriction to object/array roots
pah Aug 11, 2014
edccda0
drop object/array root restriction from recursive parser
pah Aug 11, 2014
c9f2715
Reader: drop object/array root restrction from iterative parser
pah Aug 11, 2014
c0c0c39
jsoncheckertest: skip "fail1.json", as it is now accepted by RapidJSON
pah Aug 11, 2014
e45557e
writertest: add some roundtrip tests for non-compound root values
pah Aug 11, 2014
c256752
readertest: drop object/array root restriction tests
pah Aug 11, 2014
8979359
error/{en,error}.h: drop unused kParseErrorDocumentRootNotObjectOrArr…
pah Aug 11, 2014
4d031d0
readme.md: reference RFC7159 instead of RFC4627
pah Aug 11, 2014
24b7976
tutorial.md: lift restriction to object/array root values
pah Aug 11, 2014
e29a452
readertest.cpp: use top-level Parse functions, instead of internal ones
pah Aug 11, 2014
3755470
BaseReaderHandler: allow overriding of Default() implementation
pah Aug 11, 2014
8341095
readertest.cpp: use CRTP to activate "ADD_FAILURE()" calls in handlers
pah Aug 11, 2014
d84c051
jsoncheckertest: add checks for iterative parser as well
pah Aug 11, 2014
453eda5
tabs to 4 spaces (following 0dbcc1cf)
pah Aug 11, 2014
6863033
readertest.cpp: remove remaining use of private functions
pah Aug 11, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
jsoncheckertest: skip "fail1.json", as it is now accepted by RapidJSON
  • Loading branch information
@pah
pah committed Aug 11, 2014
commit c0c0c398b6f9c630323216775dbf08b775dbce63
7 changes: 5 additions & 2 deletions test/unittest/jsoncheckertest.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,8 @@ TEST(JsonChecker, Reader) {

// jsonchecker/failXX.json
for (int i = 1; i <= 33; i++) {
if (i == 1) // fail1.json is valid in rapidjson, which has no limitation on type of root element (RFC 7159).
continue;
if (i == 18) // fail18.json is valid in rapidjson, which has no limitation on depth of nesting.
continue;

Expand All @@ -57,14 +59,15 @@ TEST(JsonChecker, Reader) {
json = ReadFile(filename, length);
if (!json) {
printf("jsonchecker file %s not found", filename);
ADD_FAILURE();
continue;
}
}

GenericDocument<UTF8<>, CrtAllocator> document; // Use Crt allocator to check exception-safety (no memory leak)
if (!document.Parse((const char*)json).HasParseError())
FAIL();
//printf("%s(%u):%s\n", filename, (unsigned)document.GetErrorOffset(), document.GetParseError());
ADD_FAILURE_AT(filename, document.GetErrorOffset());

free(json);
}

Expand Down