[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: mdb_cursor_del

David Barbour wrote:



On Wed, Feb 4, 2015 at 7:13 PM, Howard Chu <hyc@symas.com
<mailto:hyc@symas.com>> wrote:


    The behavior wasn't settled, in the beginning. But you can read the
    mtest source code to see what the expected behavior is.

    In an open source project, you are expected to read source code to
    understand what's going on.


I have been.

Though, in this particular case, I needed to dig all the way down to
LMDB page layout to grok key indexes, compare to mdb_cursor_next, search
for instances of C_DEL, etc. to understand what was happening.


You just needed to read mtest.c to understand the intended behavior.


    Things have settled down enough now to formally document it though.



Then make it so. It will save a lot of people a lot of time. :)


Suggest the language for such a change. Patch welcome.

--
  -- Howard Chu
  CTO, Symas Corp.           http://www.symas.com
  Director, Highland Sun     http://highlandsun.com/hyc/
  Chief Architect, OpenLDAP  http://www.openldap.org/project/