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.

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

Fix the Angstrom.Buffered.state_to_option docstring #164

Merged
merged 1 commit into from
Mar 3, 2019

Conversation

anmonteiro
Copy link
Contributor

@anmonteiro anmonteiro commented Feb 24, 2019

The docstring claimed to return Some (bs, v) when in reality it returns Some v.

Refs:

angstrom/lib/angstrom.ml

Lines 118 to 120 in 3a3e245

let state_to_option = function
| Done(_, v) -> Some v
| _ -> None

The docstring claimed to return `Some (bs, v)` when in reality it returns `Some v`
@seliopou seliopou merged commit 48b901c into inhabitedtype:master Mar 3, 2019
@seliopou
Copy link
Member

seliopou commented Mar 3, 2019

Thanks!

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