Updated patch

[Havoc Pennington]

On Mon, 2005-12-05 at 13:33 +0000, Matthew Johnson wrote:
> 
> 8853: assertion failed "(*(const char*)_DBUS_FUNCTION_NAME) != '_'" file "dbus-connection.c" line 2877 function _dbus_connection_read_write_dispatch
> 
> Which appears to enforce that _dbus_return_val_if_fail can only be
> called from functions that don't start with an underscore. I have no
> idea why this is, but have accordingly moved the calls to that into the
> wrapper functions.

The underscore functions are internal/private, and return_if_fail should
only be used to check correctness of arguments to public API. For
private API, just use dbus_assert() (and be sure public API that calls
the private API has return_if_fail so we don't hit the assert when
checks are enabled)

Havoc