mirage / ocaml-vchan

Pure OCaml implementation of the "vchan" shared-memory communication protocol
ISC License
45 stars 26 forks source link

Use result #130

Closed hannesm closed 5 years ago

hannesm commented 5 years ago

this fixes #113 (and is on top of #129) -- though it changes the API (Location.of_order and Port.of_string now use result, thus reverse dependencies will have to be adapted).

hannesm commented 5 years ago

breaks API, but IMHO towards better (using pre-defined result type instead of custom ones, making it more consistent)

hannesm commented 5 years ago

since #134 as well breaks API, I plan to merge this together with #134 in a release cycle (bumping the major version). please raise your voice if you oppose this.

hannesm commented 5 years ago

integrated into #134