Changeset 1093:44f3e48dfe27 in freeDiameter for libfdcore/endpoints.c

Timestamp:

May 6, 2013, 5:33:22 PM (11 years ago)

Author:

Sebastien Decugis <sdecugis@freediameter.net>

Branch:

default

Phase:

public

Message:

Align the behavior of all fd_*dump functions wrt final \n

File:

• libfdcore/endpoints.c (modified) (2 diffs)

libfdcore/endpoints.c

@@ -287 +287 @@
 DECLARE_FD_DUMP_PROTOTYPE(fd_ep_dump_one, struct fd_endpoint * ep )
 {
-        size_t o = 0;
-        if (!offset)
-                offset=&o;
+        FD_DUMP_HANDLE_OFFSET();
         
         CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "{ep}(@%p): ", ep), return NULL);
         
         if (!ep) {
-                CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "INVALID/NULL\n"), return NULL);
+                CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "INVALID/NULL"), return NULL);
                 return *buf;
         }
@@ -311 +309 @@
 {
         struct fd_list * li;
-        size_t o = 0;
-        if (!offset)
-                offset=&o;
-        
-        CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "%*s{eps}(@%p):\n", indent, "", eps), return NULL);
+        
+        FD_DUMP_HANDLE_OFFSET();
+        
+        CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "%*s{eps}(@%p):", indent, "", eps), return NULL);
         if (eps) {
                 for (li = eps->next; li != eps; li = li->next) {
                         struct fd_endpoint * ep = (struct fd_endpoint *)li;
-                        CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "%*s", indent+1, ""), return NULL);
+                        CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "\n%*s", indent+1, ""), return NULL);
                         CHECK_MALLOC_DO( fd_ep_dump_one( FD_DUMP_STD_PARAMS, ep ), return NULL);
-                        CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "\n"), return NULL);
-                }
-        }
-}
-        
+                }
+        }
+}
+