diff options
author | Havoc Pennington <hp@redhat.com> | 2005-01-17 00:16:28 +0000 |
---|---|---|
committer | Havoc Pennington <hp@redhat.com> | 2005-01-17 00:16:28 +0000 |
commit | aad6fa897f85486386b030847151cb09943c97c0 (patch) | |
tree | 3996469d2be2df0bc63a4e8dd0aa3ac6cdfaddcb /dbus/dbus-connection.c | |
parent | 7f9721a6d36697a57170c604aa845af3531bd157 (diff) |
2005-01-16 Havoc Pennington <hp@redhat.com>
* dbus/dbus-internals.c (_dbus_real_assert): print the function
name the assertion failed in
* dbus/dbus-internals.h (_dbus_return_if_fail)
(_dbus_return_val_if_fail): assert that the name of the function
containing the check doesn't start with '_', since we only want to
use checks on public functions
* dbus/dbus-connection.c (_dbus_connection_ref_unlocked): change
checks to assertions
* dbus/dbus-marshal-header.c (_dbus_header_set_field_basic):
change checks to asserts for private function
* dbus/dbus-message.c (_dbus_message_set_serial): checks
to asserts for private function
* dbus/dbus-marshal-recursive.c (skip_one_complete_type): remove
broken assertion that was breaking make check
(_dbus_type_reader_array_is_empty): remove this rather than fix
it, was only used in assertions
Diffstat (limited to 'dbus/dbus-connection.c')
-rw-r--r-- | dbus/dbus-connection.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/dbus/dbus-connection.c b/dbus/dbus-connection.c index 197d186c..d338b786 100644 --- a/dbus/dbus-connection.c +++ b/dbus/dbus-connection.c @@ -1004,8 +1004,8 @@ _dbus_connection_new_for_transport (DBusTransport *transport) DBusConnection * _dbus_connection_ref_unlocked (DBusConnection *connection) { - _dbus_return_val_if_fail (connection != NULL, NULL); - _dbus_return_val_if_fail (connection->generation == _dbus_current_generation, NULL); + _dbus_assert (connection != NULL); + _dbus_assert (connection->generation == _dbus_current_generation); #ifdef DBUS_HAVE_ATOMIC_INT _dbus_atomic_inc (&connection->refcount); @@ -1442,7 +1442,7 @@ _dbus_connection_preallocate_send_unlocked (DBusConnection *connection) { DBusPreallocatedSend *preallocated; - _dbus_return_val_if_fail (connection != NULL, NULL); + _dbus_assert (connection != NULL); preallocated = dbus_new (DBusPreallocatedSend, 1); if (preallocated == NULL) @@ -1897,9 +1897,9 @@ _dbus_connection_block_for_reply (DBusConnection *connection, long tv_sec, tv_usec; DBusDispatchStatus status; - _dbus_return_val_if_fail (connection != NULL, NULL); - _dbus_return_val_if_fail (client_serial != 0, NULL); - _dbus_return_val_if_fail (timeout_milliseconds >= 0 || timeout_milliseconds == -1, FALSE); + _dbus_assert (connection != NULL); + _dbus_assert (client_serial != 0); + _dbus_assert (timeout_milliseconds >= 0 || timeout_milliseconds == -1); if (timeout_milliseconds == -1) timeout_milliseconds = _DBUS_DEFAULT_TIMEOUT_VALUE; |