search

ocaml-community / zed

Abstract engine for text edition in OCaml
Other
121 stars 16 forks source link

`Zed_rope.Zip.make_b` is broken #28

Closed Drup closed 4 years ago

Drup commented 4 years ago 
# let z = Zed_rope.of_string @@ Zed_string.of_utf8 "abc" ;;
val z : Zed_rope.t = <abstr>
# let zip = Zed_rope.Zip.make_b z 0 ;; (* Still works *)
val zip : Z.t = <abstr>
# let zip = Zed_rope.Zip.make_b z 1 ;; (* Used to work before 2.0 *)
Exception: Zed_string.Out_of_bounds.
# let zip = Zed_rope.Zip.make_b z (-1) ;; (* Doesn't work either *)
Exception: Zed_rope.Out_of_bounds.
kandu commented 4 years ago

Thanks for reporting, @Drup. I'll cut a bugfix release after all the CI checks pass.