Remove trailing newlines from _dbus_assert_not_reached
authorSimon McVittie <smcv@debian.org>
Thu, 21 Jul 2016 09:59:50 +0000 (10:59 +0100)
committerSimon McVittie <simon.mcvittie@collabora.co.uk>
Fri, 12 Aug 2016 16:11:48 +0000 (17:11 +0100)
This was never necessary: _dbus_assert_not_reached() always added one.

Signed-off-by: Simon McVittie <smcv@debian.org>
Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=97009

bus/bus.c
bus/config-parser-trivial.c
bus/config-parser.c
dbus/dbus-auth-util.c
dbus/dbus-dataslot.c
dbus/dbus-message-util.c
dbus/dbus-object-tree.c
dbus/dbus-string-util.c
test/break-loader.c
test/test-service.c

index 9996be2..103447c 100644 (file)
--- a/bus/bus.c
+++ b/bus/bus.c
@@ -1699,7 +1699,7 @@ bus_context_check_security_policy (BusContext     *context,
         }
       else
         {
-          _dbus_assert_not_reached ("a message was somehow sent to an inactive recipient from a source other than the message bus\n");
+          _dbus_assert_not_reached ("a message was somehow sent to an inactive recipient from a source other than the message bus");
           recipient_policy = NULL;
         }
     }
index 03ad838..0ba5f36 100644 (file)
@@ -497,7 +497,7 @@ process_test_valid_subdir (const DBusString *test_base_dir,
   dir = NULL;
 
   if (!_dbus_string_init (&test_directory))
-    _dbus_assert_not_reached ("didn't allocate test_directory\n");
+    _dbus_assert_not_reached ("didn't allocate test_directory");
 
   _dbus_string_init_const (&filename, subdir);
 
@@ -510,7 +510,7 @@ process_test_valid_subdir (const DBusString *test_base_dir,
 
   _dbus_string_free (&filename);
   if (!_dbus_string_init (&filename))
-    _dbus_assert_not_reached ("didn't allocate filename string\n");
+    _dbus_assert_not_reached ("didn't allocate filename string");
 
   dbus_error_init (&error);
   dir = _dbus_directory_open (&test_directory, &error);
index 2e2fa23..15d4228 100644 (file)
@@ -2942,7 +2942,7 @@ process_test_valid_subdir (const DBusString *test_base_dir,
   dir = NULL;
 
   if (!_dbus_string_init (&test_directory))
-    _dbus_assert_not_reached ("didn't allocate test_directory\n");
+    _dbus_assert_not_reached ("didn't allocate test_directory");
 
   _dbus_string_init_const (&filename, subdir);
 
@@ -2955,7 +2955,7 @@ process_test_valid_subdir (const DBusString *test_base_dir,
 
   _dbus_string_free (&filename);
   if (!_dbus_string_init (&filename))
-    _dbus_assert_not_reached ("didn't allocate filename string\n");
+    _dbus_assert_not_reached ("didn't allocate filename string");
 
   dbus_error_init (&error);
   dir = _dbus_directory_open (&test_directory, &error);
index 10cbc49..3675884 100644 (file)
@@ -51,7 +51,7 @@ process_test_subdir (const DBusString          *test_base_dir,
   dir = NULL;
   
   if (!_dbus_string_init (&test_directory))
-    _dbus_assert_not_reached ("didn't allocate test_directory\n");
+    _dbus_assert_not_reached ("didn't allocate test_directory");
 
   _dbus_string_init_const (&filename, subdir);
   
@@ -64,7 +64,7 @@ process_test_subdir (const DBusString          *test_base_dir,
 
   _dbus_string_free (&filename);
   if (!_dbus_string_init (&filename))
-    _dbus_assert_not_reached ("didn't allocate filename string\n");
+    _dbus_assert_not_reached ("didn't allocate filename string");
 
   dir = _dbus_directory_open (&test_directory, &error);
   if (dir == NULL)
index e37c9dd..a4b3bfc 100644 (file)
@@ -394,7 +394,7 @@ _dbus_data_slot_test (void)
       _dbus_data_slot_allocator_alloc (&allocator, &tmp);
 
       if (tmp != i)
-        _dbus_assert_not_reached ("did not allocate slots in numeric order\n");
+        _dbus_assert_not_reached ("did not allocate slots in numeric order");
 
       ++i;
     }
index a9d4a51..1096b45 100644 (file)
@@ -571,7 +571,7 @@ process_test_subdir (const DBusString          *test_base_dir,
   dir = NULL;
 
   if (!_dbus_string_init (&test_directory))
-    _dbus_assert_not_reached ("didn't allocate test_directory\n");
+    _dbus_assert_not_reached ("didn't allocate test_directory");
 
   _dbus_string_init_const (&filename, subdir);
 
@@ -584,7 +584,7 @@ process_test_subdir (const DBusString          *test_base_dir,
 
   _dbus_string_free (&filename);
   if (!_dbus_string_init (&filename))
-    _dbus_assert_not_reached ("didn't allocate filename string\n");
+    _dbus_assert_not_reached ("didn't allocate filename string");
 
   dir = _dbus_directory_open (&test_directory, &error);
   if (dir == NULL)
index 27481a8..22565ef 100644 (file)
@@ -1646,7 +1646,7 @@ run_decompose_tests (void)
               ++i;
             }
           _dbus_warn ("}\n");
-          _dbus_assert_not_reached ("path decompose failed\n");
+          _dbus_assert_not_reached ("path decompose failed");
         }
 
       dbus_free_string_array (result);
index 5b09a69..85bc570 100644 (file)
@@ -312,12 +312,12 @@ _dbus_string_test (void)
   while (i < 10)
     {
       if (!_dbus_string_append (&str, "a"))
-        _dbus_assert_not_reached ("failed to append string to string\n");
+        _dbus_assert_not_reached ("failed to append string to string");
 
       _dbus_assert (_dbus_string_get_length (&str) == i * 2 + 1);
 
       if (!_dbus_string_append_byte (&str, 'b'))
-        _dbus_assert_not_reached ("failed to append byte to string\n");
+        _dbus_assert_not_reached ("failed to append byte to string");
 
       _dbus_assert (_dbus_string_get_length (&str) == i * 2 + 2);
                     
index 1e406fc..dca6e42 100644 (file)
@@ -509,10 +509,10 @@ find_breaks_based_on (const DBusString   *filename,
   retval = FALSE;
 
   if (!_dbus_string_init (&orig_data))
-    _dbus_assert_not_reached ("could not allocate string\n");
+    _dbus_assert_not_reached ("could not allocate string");
 
   if (!_dbus_string_init (&mutated))
-    _dbus_assert_not_reached ("could not allocate string\n");
+    _dbus_assert_not_reached ("could not allocate string");
 
   if (!dbus_internal_do_not_use_load_message_file (filename, is_raw,
                                                    &orig_data))
index c0bd2c6..564875b 100644 (file)
@@ -112,7 +112,7 @@ check_hello_from_self_reply (DBusPendingCall *pcall,
       dbus_error_free (&error);
     }
   else
-     _dbus_assert_not_reached ("Unexpected message received\n");
+     _dbus_assert_not_reached ("Unexpected message received");
 
   hello_from_self_reply_received = TRUE;