Drup / ocaml-lmdb

Ocaml bindings for lmdb.
https://drup.github.io/ocaml-lmdb/dev/
MIT License
48 stars 3 forks source link

Move Lmdb.Map.Conv to Lmdb.Conv #24

Closed madroach closed 5 years ago

madroach commented 5 years ago

Which is better? Lmdb.Map.Conv, so that one will write

Map.(open_existing Nodup ~key:Conv.string ~value:Conv.string) env

or Lmdb.Conv, so that one may also write

Map.open_existing Nodup ~key:Conv.string ~value:Conv.string env
Drup commented 5 years ago

the later, so that one can write

Lmdb.(Map.open_existing Nodup ~key:Conv.string ~value:Conv.string) env

The user should not open Lmdb.