index.delete requires unique ids and errors not thrown

per http://toblerity.org/rtree/class.html#rtree.index.Index.delete it states for the id parameter to delete: long integer A long integer that is the identifier for this index entry. IDs need not be unique to be inserted into the index, and it is up to the user to ensure they are unique if this is a requirement.

However I tried this with with an Index which had a set of points which all had the same ids and it never removed it items. As soon as I gave each point a unique id the points got removed.

There are two bugs here:

1. documentation should be updated or implementation fixed
2. the python code should be checking error code returned by Index_DeleteData and throwing exception if the delete failed. Probably should go over the whole index.py file to make sure returned error codes are enforced.

Read the Index.delete documentation at the linked API page and inspect index.py, starting with Index_DeleteData and its handling of returned error codes. Reproduce deletion with duplicate and unique IDs, then align the documentation or implementation and ensure failed deletions are reported consistently across the mentioned file.