deprecate duphandle - Githubissues

ygrek / ocurl

OCaml bindings to libcurl

https://ygrek.org/p/ocurl

MIT License

59 stars 32 forks source link

Closed ygrek closed 5 years ago

ygrek commented 6 years ago

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 6 years ago

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 5 years ago

going to remove in next release