Raise correct exceptions in abortVersion() and commitVersion().

Spread out some code that had the vertical white space removed.

Fix the versions() implementation so that it works if the optional max
argument is not passed by the caller.

Fix commitVersion() so that it works for committing from one version
to the other.  The previous version always committed to the
non-version data even if a destination version was specified.

Fix typos.

documented exceptions.

Remove a few of the old #JF# comments that explain why some old,
commented out test code didn't work.  XXX still need to do more.

Extend a few tests with more checks of accessing non-version data via
versions that have no changes in them.

Comment -> doc string.

It's possible for a storage to not support historical revisions, in
which case this test can't work.

Also make comment at top of BasicStorage into a doc string.

Replace a string-based exception (a string literal which was broken
even for string-based exceptions) with a class-based exception.

Raise a VersionCommitError is source and dest versions were the same.

way all the other MinPO objects are created.  But, be sure to pass
already_pickled=1 to _dostore().

of two objects.  This uncovered a bug in Berkeley's CommitLog, which
was caused by an inadvertently committed change.

Changed the conflict resolution machinery to avoid attempting to resolve conflicts on ZClasses, because we can't import their "modules" (we don't have a db connection).

actual work

database file not to be truncated when it should have been during a
two-phase commit abort.

Simplify santiy checking of length fields in transaction metadata.
Replace the following test:
[1]    if ul > tl or dl > tl or el > tl or tl < (23+ul+dl+el):
with this one:
[2]    if tl < (23+ul+dl+el):

If any of the first three comparisons in [1] is true, then the final
test must also be true.  It's unlikely that any of the tests will be
true, so just do the final one.

This test wasn't part of read_index(), but seems good and cheap.

XXX Needs test case.

The only exceptions that an _p_resolveConflict() method should raise
are ConflictError and ValueError.  If it raises a different exception,
then it has failed.

In general, a unittest should not use a bare except clause, because
they almost always mask failures.  If the test really means to catch
all exceptions, it's best to make a comment to that effect.

self.assertEqual() and friends since this gives much better output.

Based on conversation with Evan yesterday, I think this is safe.

FCNTL is deprecated in Python 2.2 and all its functionality is
available in fcntl in Python 2.1.

Handle two new failure conditions by ignoring the index and
continuing:

    - corrupted pickle in .index file
    - no value for 'pos' in pickle (used to call long(None))

bucket when doing a "low" search.

Remove Python 2.x-isms so that tests run under Python 1.5.2

Use asyncwrap to call asyncore.loop() and asyncore.poll().
Add definition for EINTR.

Shorten two long lines, one by wrapping, another by removing an
unnecessary tuple unpacking.

Wrap send() and recv() calls in try/except

The try/except catches errors like EAGAIN that indicate transient
failures.  In those cases, treat as send() and recv() of no data and
return from method gracefully.