Currently, bson_return_if_fail() is used in many places to validate user input (e.g. non-null pointers, ranges of integer arguments); however, these macros are compiled out of non-debug builds.

For basic conditional checks, which aren't expensive, do we want those to remain in production builds?