Fix bug in split position handling
commitedc381036725373818fa04d137a69f5941a95150
authorOlly Betts <olly@survex.com>
Tue, 30 Oct 2018 15:58:51 +0000 (31 04:58 +1300)
committerOlly Betts <olly@survex.com>
Tue, 30 Oct 2018 15:58:51 +0000 (31 04:58 +1300)
tree853927ba9a6de2a3a0826f4b21ed3598c616651e
parentbcb093ba7f73d59337943e5064fdb1af4526ebbe
Fix bug in split position handling

When we look to see in the new position is present before the split
we were only checking if the std::lower_bound() (binary chop) ended
up on a valid position, but we also need to check if that position
is the one we're trying to add.

This was leading to positions getting lost in this situation.
xapian-core/api/terminfo.cc
xapian-core/tests/api_backend.cc