[C++][Integration] Refactor json_internal.cc to use fewer `DCHECK`s
Open
- Dominant language
- C++
- Stars
- 17.1k
- Forks
- 4.3k
- Avg merge
- 3d 13h
- Merged PRs (30d)
- 88
Description
`json_internal.cc` uses `DCHECK`s to check many conditions even in scopes where a status could be returned instead. These should be replaced with error statuses, which will help pinpoint invalid JSON more explicitly
Contributor guide
Assessment
This issue has not been assessed yet.