elm-community / parser-combinators

A parser combinator library for Elm.

Home Page:http://package.elm-lang.org/packages/elm-community/parser-combinators/latest

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Why not use Result from core?

igrep opened this issue · comments

Why is the type Combine.Result defined as

type Result res
  = Done res
  | Fail (List String)

instead of

type alias Result res = Result.Result (List String) res

using the Result.Result from the core even though they are obviously isomorphic?
It's annoying to convert every time calling parse!

I'll make this change in the coming weeks. The reason I went with a new ADT initially is because I thought I would end up creating more result states that just those two.