Skip to content

Tail recursive parse#1225

Merged
cartermp merged 4 commits intofsprojects:masterfrom
eugbaranov:feature/tail-recursive-parse
Mar 3, 2021
Merged

Tail recursive parse#1225
cartermp merged 4 commits intofsprojects:masterfrom
eugbaranov:feature/tail-recursive-parse

Conversation

@eugbaranov
Copy link
Copy Markdown
Contributor

Rewrote a few functions in CPS allowing for deeper structures without stack overflow.

@dsyme
Copy link
Copy Markdown
Contributor

dsyme commented Mar 21, 2019

Close/open to re-run CI

@dsyme dsyme closed this Mar 21, 2019
@dsyme dsyme reopened this Mar 21, 2019
@eugbaranov eugbaranov force-pushed the feature/tail-recursive-parse branch from 39188e2 to 7a340b4 Compare June 18, 2019 22:52
@dsyme dsyme closed this Oct 4, 2020
@dsyme dsyme reopened this Oct 4, 2020
@eugbaranov eugbaranov force-pushed the feature/tail-recursive-parse branch from 7a340b4 to 5ce9ed7 Compare October 5, 2020 09:26
@eugbaranov eugbaranov force-pushed the feature/tail-recursive-parse branch from 5ce9ed7 to 2ed8ec8 Compare October 7, 2020 10:40
@cartermp cartermp closed this Mar 3, 2021
@cartermp cartermp reopened this Mar 3, 2021
Copy link
Copy Markdown
Collaborator

@cartermp cartermp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the improvements! I will merge this.

@cartermp
Copy link
Copy Markdown
Collaborator

cartermp commented Mar 3, 2021

N.B. there is a weird artifact of the old appveyor webhook here. It can be ignored.

@cartermp cartermp merged commit f117c4b into fsprojects:master Mar 3, 2021
Numpsy added a commit to Numpsy/Jay that referenced this pull request Aug 24, 2025
…ack overflows

Imports the changes in

fsprojects/FSharp.Data#1225
fsprojects/FSharp.Data#1411

from FSharp.Data, to allow parsing of more deeply nested Json structures without a stack overflow
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.

3 participants