Navigation

Changeset 1093:44f3e48dfe27 in freeDiameter for libfdcore


Ignore:
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

Location:
libfdcore
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • libfdcore/config.c

    r1085 r1093  
    8383DECLARE_FD_DUMP_PROTOTYPE(fd_conf_dump)
    8484{
    85         size_t o=0;
    86         if (!offset)
    87                 offset = &o;
     85        FD_DUMP_HANDLE_OFFSET();
    8886       
    8987        CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "{freeDiameter configuration}(@%p): \n", fd_g_config), return NULL);       

  • libfdcore/endpoints.c

    r1085 r1093  
    287287DECLARE_FD_DUMP_PROTOTYPE(fd_ep_dump_one, struct fd_endpoint * ep )
    288288{
    289         size_t o = 0;
    290         if (!offset)
    291                 offset=&o;
     289        FD_DUMP_HANDLE_OFFSET();
    292290       
    293291        CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "{ep}(@%p): ", ep), return NULL);
    294292       
    295293        if (!ep) {
    296                 CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "INVALID/NULL\n"), return NULL);
     294                CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "INVALID/NULL"), return NULL);
    297295                return *buf;
    298296        }
    … …  
    311309{
    312310        struct fd_list * li;
    313         size_t o = 0;
    314         if (!offset)
    315                 offset=&o;
    316        
    317         CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "%*s{eps}(@%p):\n", indent, "", eps), return NULL);
     311       
     312        FD_DUMP_HANDLE_OFFSET();
     313       
     314        CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "%*s{eps}(@%p):", indent, "", eps), return NULL);
    318315        if (eps) {
    319316                for (li = eps->next; li != eps; li = li->next) {
    320317                        struct fd_endpoint * ep = (struct fd_endpoint *)li;
    321                         CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "%*s", indent+1, ""), return NULL);
     318                        CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "\n%*s", indent+1, ""), return NULL);
    322319                        CHECK_MALLOC_DO( fd_ep_dump_one( FD_DUMP_STD_PARAMS, ep ), return NULL);
    323                         CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "\n"), return NULL);
    324                 }
    325         }
    326 }
    327        
     320                }
     321        }
     322}
     323       

  • libfdcore/events.c

    r1085 r1093  
    165165{
    166166        struct fd_list * li;
    167         size_t o=0;
    168         if (!offset)
    169                 offset=&o;
     167        FD_DUMP_HANDLE_OFFSET();
    170168       
    171169        CHECK_POSIX_DO( pthread_rwlock_rdlock(&trig_rwl),  );
    … …  
    173171        for (li = trig_list.next; li != &trig_list; li = li->next) {
    174172                struct trig_item *t = li->o;
    175                 CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "{trigger sig:%d}'%s'->%p ", t->trig_value, t->trig_module, t->cb), break);
     173                CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "{signal:%d}'%s'->%p ", t->trig_value, t->trig_module, t->cb), break);
    176174        }
    177175       

  • libfdcore/extensions.c

    r1085 r1093  
    8888        {
    8989                struct fd_ext_info * ext = (struct fd_ext_info *)li;
    90                 CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "{extension}(@%p): '%s'[%s], %sloaded\n", ext, ext->filename, ext->conffile?:"no conf", ext->handler ? "" : "not "), return NULL);
     90                CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "{extension}(@%p): '%s'[%s], %sloaded%s", ext, ext->filename, ext->conffile?:"no conf", ext->handler ? "" : "not ", (li->next == &ext_list) ? "":"\n"), return NULL);
    9191        }
    9292        return *buf;

  • libfdcore/peers.c

    r1085 r1093  
    412412DECLARE_FD_DUMP_PROTOTYPE(fd_peer_dump, struct peer_hdr * p, int details)
    413413{
    414         size_t o=0;
    415         if (!offset)
    416                 offset = &o;
     414        FD_DUMP_HANDLE_OFFSET();
    417415       
    418416        CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "{peer}(@%p): ", p), return NULL);
    … …  
    454452{
    455453        struct fd_list * li;
    456         size_t o=0;
    457         if (!offset)
    458                 offset = &o;
     454        FD_DUMP_HANDLE_OFFSET();
    459455       
    460456        CHECK_FCT_DO( pthread_rwlock_rdlock(&fd_g_peers_rw), /* continue */ );
    … …  
    462458        for (li = fd_g_peers.next; li != &fd_g_peers; li = li->next) {
    463459                CHECK_MALLOC_DO( fd_peer_dump(FD_DUMP_STD_PARAMS, (struct peer_hdr *)li->o, details), break);
    464                 CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "\n"), break);
     460                if (li->next != &fd_g_peers) {
     461                        CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "\n"), break);
     462                }
    465463        }
    466464       

  • libfdcore/server.c

    r1085 r1093  
    9797{
    9898        struct fd_list * li, *cli;
    99         size_t o=0;
    100         if (!offset)
    101                 offset = &o;
     99       
     100        FD_DUMP_HANDLE_OFFSET();
    102101       
    103102        for (li = FD_SERVERS.next; li != &FD_SERVERS; li = li->next) {
    … …  
    105104                enum s_state st = get_status(s);
    106105               
    107                 CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "{server}(@%p)'%s': %s, %s, %s\n", s, fd_cnx_getid(s->conn),
     106                CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "{server}(@%p)'%s': %s, %s, %s", s, fd_cnx_getid(s->conn),
    108107                                IPPROTO_NAME( s->proto ),
    109108                                s->secur ? "Secur" : "NotSecur",
    … …  
    117116                        struct client * c = (struct client *)cli;
    118117                        char bufts[128];
    119                         CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "  {client}(@%p)'%s': to:%s\n", c, fd_cnx_getid(c->conn), fd_log_time(&c->ts, bufts, sizeof(bufts))), break);
     118                        CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "\n  {client}(@%p)'%s': to:%s", c, fd_cnx_getid(c->conn), fd_log_time(&c->ts, bufts, sizeof(bufts))), break);
    120119                }
    121120                CHECK_POSIX_DO( pthread_mutex_unlock(&s->clients_mtx), );
     121               
     122                if (li->next != &FD_SERVERS) {
     123                        CHECK_MALLOC_DO( fd_dump_extend( FD_DUMP_STD_PARAMS, "\n"), return NULL);
     124                }
    122125        }
    123126       
Note: See TracChangeset for help on using the changeset viewer.