migrating from bson_return_* to BSON_ASSERT

bson_return_*_if_fail never made much sense.  Just use straight asserts