ygrek/ocurl

deprecate duphandle

ygrek opened this issue · 2 comments

ygrek commented

See #30 for motivation.
Rationale:

  • it is assumed to be used very infrequently if at all
  • supporting duphandle requires keeping references to all ocaml values passed in setopts - and this price is paid by users not using duphandle at all (which is supposed to be vast majority)
  • it also puts extra burden on adding every new setopt
  • it is pretty easy to replicate its functionality explicitly on ocaml side just by grouping all setopts into a list or closure and applying on fresh handle
nojb commented

FWIW, I am strongly in favour of removing duphandle. Not necessary for a language like OCaml and adds a lot of complexity to the bindings.

ygrek commented

going to remove in next release