Howard Chu authored 10 years ago and Quanah Gibson-Mount committed 10 years ago

Check for duplicate scopes in search_aliases, not later.

Howard Chu authored 10 years ago and Quanah Gibson-Mount committed 10 years ago

This reverts commit cac00c6a.

Howard Chu authored 10 years ago and Quanah Gibson-Mount committed 10 years ago

Aliases pointing within the original search scope could cause
the same entry to be returned multiple times. Check for subtree
scope and duplicate scopes.

we already checked for data == NULL, drop unnecessary if

Move CACHEFLUSH, avoid unnecessary if()

Howard Chu authored 10 years ago and Quanah Gibson-Mount committed 10 years ago

Must save/restore mcd cursor if we're doing a scope-based search.

Howard Chu authored 10 years ago and Quanah Gibson-Mount committed 10 years ago

Harmless, but stupid.

Of course, MacOSX only has posix_memalign, and only in recent versions.
(Older versions could use valloc instead. That precludes ever changing
the LMDB pagesize to be different from the OS pagesize.) Not all that
crucial since it only affects mdb_env_copyfd1.

Leaving them in dirty_list caused breakage.  Instead merge them
into me_pghead at the end, when no more pages will be allocated.

    This needs to be streamlined in 2.5, current tool API is a mess.

Howard Chu authored 10 years ago and Quanah Gibson-Mount committed 10 years ago

Don't accept config directives for cache DB until the cache backend type
has been set.

Howard Chu authored 10 years ago and Quanah Gibson-Mount committed 10 years ago

db_open/close were not symmetric with init/destroy

Don't let dummy go out of scope

Everything has it. Solaris doesn't have posix_memalign.

Howard Chu authored 10 years ago and Quanah Gibson-Mount committed 10 years ago

rename ppolicy_close -> ppolicy_db_close for consistency

Howard Chu authored 10 years ago and Quanah Gibson-Mount committed 10 years ago

don't free pwcons unless we're actually destroying this DB