Skip to content

various minor changes to fractured JSON support#2591

Merged
FranciscoThiesen merged 1 commit intofrancisco/fractured_json_supportfrom
lemire/fractured_json_support
Jan 20, 2026
Merged

various minor changes to fractured JSON support#2591
FranciscoThiesen merged 1 commit intofrancisco/fractured_json_supportfrom
lemire/fractured_json_support

Conversation

@lemire
Copy link
Member

@lemire lemire commented Jan 17, 2026

This is a PR on top of @FranciscoThiesen's PR regarding Fractured JSON.

  • I am removing two functions that do not appear important.
  • I am adding some documentation
  • I am tweaking the tests (and other bits) so that they use our API is the 'proper' manner: we avoid value() and error() as it is error prone.

@FranciscoThiesen
Copy link
Member

LGTM!

@FranciscoThiesen FranciscoThiesen merged commit 5b6e723 into francisco/fractured_json_support Jan 20, 2026
147 checks passed
@FranciscoThiesen FranciscoThiesen deleted the lemire/fractured_json_support branch January 20, 2026 03:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants