Skip to content

Commit

Permalink
Merge pull request #222 from inhabitedtype/remove-result-dep
Browse files Browse the repository at this point in the history
remove references to result package
  • Loading branch information
seliopou authored Oct 27, 2022
2 parents 94ea5d6 + 8a7105a commit eb2cb26
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 6 deletions.
1 change: 0 additions & 1 deletion httpaf.opam
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ depends: [
"bigstringaf" {>= "0.4.0"}
"angstrom" {>= "0.14.0"}
"faraday" {>= "0.6.1"}
"result"
]
synopsis:
"A high-performance, memory-efficient, and scalable web server for OCaml"
Expand Down
2 changes: 1 addition & 1 deletion lib/dune
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,5 @@
(name httpaf)
(public_name httpaf)
(libraries
angstrom faraday bigstringaf result)
angstrom faraday bigstringaf)
(flags (:standard -safe-string)))
3 changes: 0 additions & 3 deletions lib/httpaf.mli
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,6 @@
1.1 specification, and the basic principles of memory management and
vectorized IO. *)


open Result

(** {2 Basic HTTP Types} *)


Expand Down
2 changes: 1 addition & 1 deletion lib/reqd.ml
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,7 @@ let report_error t error =
let report_exn t exn =
report_error t (`Exn exn)

let try_with t f : (unit, exn) Result.result =
let try_with t f : (unit, exn) result =
try f (); Ok () with exn -> report_exn t exn; Error exn

(* Private API, not exposed to the user through httpaf.mli *)
Expand Down

0 comments on commit eb2cb26

Please sign in to comment.