Lorenz Bauer authored Oct 20, 2016 and Hallvard Furuseth committed Dec 13, 2016

Return EPIPE instead.

Don't use it as a "cursor is tracked" hint in mdb_pages_xkeep().
It's been harmless so far, but would break after mdb_cursor_copy().
Checking m0 directly short-circuits better anyway.

Drop ~C_EOF, pointless after 8c215aa9

Other functions depend on them to do so.
For mdb_node_read(), instead remove such a dependence.

Init me_rpmutex independently of MDB_NOLOCK.
Plug leaks on mdb_env_open() failure.
Tweak mdb_env_close0() to handle the rearranged mdb_env_open().

xcursor fixup depends on init state

Revert excess cursor fixup

No change in functionality.

No change in functionality, except needs less mallocing.

The 0xFFFD check seems due to misleading MultiByteToWideChar() doc.
Bad UTF-8 gives 0xFFFD in the output string, not the return value.

...and check !MDB_CLOEXEC in an 'if' rather than '#if'
to match its non-zero usage.

We can run without F_NOCACHE if it fails. And we do not know
what combining it with O_DIRECT means, if a system has both.

1st struct member was not a scalar.

It usually gets done anyway, but one of the fastpath shortcuts
bypassed this step.

Make explicit and default nonzero equivalent. Parenthesize.

mdb_cursor_set wasn't unref'ing as intended.

Drop macro Y=MDB_FMT_Y, add Yu/Yd=MDB_PRIy(). Replace
Y"d..." -> Yd"...", Y"u..." -> Yu"..." / MDB_SCNy(u)"...".

When using format modifier "ll" or "I64", use the matching
type unsigned <long long / __int64> rather than uint64_t.

This matches the mdb_size_t entry counts.