dpwiz / hedn / issues / #3 - parseMaybe and decode ignore unparsed leftovers — Bitbucket

Issue #3 resolved

Yuriy Syrovetskiy created an issue 2018-11-27

λ> parseMaybe "DO NOT"
    Just (NoTag (Symbol "" "DO"))

Comments (7)

Yuriy Syrovetskiy reporter
- edited description
- 2018-11-27T16:17:20+00:00
dp wiz repo owner
That's that attoparsec does in maybeResult.
- 2018-12-03T13:20:38+00:00

Yuriy Syrovetskiy reporter

But aeson doesn't.

λ> EDN.decode "DO NOT" :: Maybe EDN.Value
    Just (Symbol "" "DO")

λ> Aeson.decode "\"DO\" \"NOT\"" :: Maybe Aeson.Value
    Nothing

That is unexpected.

2018-12-03T13:28:32+00:00

Yuriy Syrovetskiy reporter
- changed title to parseMaybe and decode ignore unparsed leftovers
- 2018-12-03T13:29:11+00:00
dp wiz repo owner
- changed status to resolved
Yep. Top-level parser had no EOF token. Fixed in #6.
- 2019-01-01T15:53:20+00:00
Yuriy Syrovetskiy reporter
Why not quick-fix it with p <* eof?
- 2019-01-01T16:47:04+00:00
Yuriy Syrovetskiy reporter
If you incorporate this problem into a bigger one, the proper status is wontfix.
- 2019-01-01T16:48:28+00:00
Log in to comment

Assignee: –

Type: bug

Priority: major

Status: resolved

Votes: 0

Watchers: 1

Jira: the preferred issue tracker for Bitbucket. Join the team!