Changeset 1c3008a


Ignore:
Timestamp:
2009-12-13T14:48:56Z (10 years ago)
Author:
Wilmer van der Gaast <wilmer@…>
Branches:
master
Children:
e8c8d00
Parents:
2288705
Message:

No functional changes, just some style "fixes". Although I admit I'm
somewhat growing out of my own coding style, I do try to keep things
consistent at least within files.

Comments are now in reviewboard:

http://code.bitlbee.org/rb/r/13/

Files:
8 edited

Legend:

Unmodified
Added
Removed
  • dcc.c

    r2288705 r1c3008a  
    115115{
    116116        file_transfer_t *file = g_new0( file_transfer_t, 1 );
    117         dcc_file_transfer_t *df = file->priv = g_new0( dcc_file_transfer_t, 1);
     117        dcc_file_transfer_t *df = file->priv = g_new0( dcc_file_transfer_t, 1 );
     118       
    118119        file->file_size = file_size;
    119120        file->file_name = g_strdup( file_name );
     
    144145        /* listen and request */
    145146
    146         if( ( df->fd = ft_listen( &saddr, host, port, TRUE, &errmsg ) ) == -1 ) {
     147        if( ( df->fd = ft_listen( &saddr, host, port, TRUE, &errmsg ) ) == -1 )
     148        {
    147149                dcc_abort( df, "Failed to listen locally, check your ft_listen setting in bitlbee.conf: %s", errmsg );
    148150                return NULL;
     
    161163        df->progress_timeout = b_timeout_add( DCC_MAX_STALL * 1000, dcc_progress, df );
    162164
    163         imcb_log( ic, "File transfer request from %s for %s (%zd kb). ", user_nick, file_name, file_size/1024 );
    164 
    165         imcb_log( ic, "Accept the file transfer if you'd like the file. If you don't, issue the 'transfers reject' command.");
     165        imcb_log( ic, "File transfer request from %s for %s (%zd kb).\n"
     166                      "Accept the file transfer if you'd like the file. If you don't, "
     167                      "issue the 'transfers reject' command.",
     168                      user_nick, file_name, file_size / 1024 );
    166169
    167170        return file;
     
    199202                /* no progress. cancel */
    200203                if( df->bytes_sent == 0 )
    201                         return dcc_abort( df, "Couldnt establish transfer within %d seconds", DCC_MAX_STALL );
     204                        return dcc_abort( df, "Couldn't establish transfer within %d seconds", DCC_MAX_STALL );
    202205                else
    203206                        return dcc_abort( df, "Transfer stalled for %d seconds at %d kb", DCC_MAX_STALL, df->bytes_sent / 1024 );
     
    230233                         ntohl( saddr_ipv4->sin_addr.s_addr ) );
    231234                port = saddr_ipv4->sin_port;
    232         } else
     235        }
     236        else
    233237        {
    234238                struct sockaddr_in6 *saddr_ipv6 = ( struct sockaddr_in6 *) saddr;
     
    250254       
    251255        if ( !irc_msgfrom( df->ic->irc, user_nick, cmd ) )
    252                 return dcc_abort( df, "couldn't send 'DCC SEND' message to %s", user_nick );
     256                return dcc_abort( df, "Couldn't send `DCC SEND' message to %s.", user_nick );
    253257
    254258        g_free( cmd );
     
    313317        short revents;
    314318       
    315         if( !dcc_poll( df, fd, &revents) )
     319        if( !dcc_poll( df, fd, &revents ) )
    316320                return FALSE;
    317321
     
    349353                int ret;
    350354               
    351                 ASSERTSOCKOP( ret = recv( fd, &bytes_received, sizeof( bytes_received  ), MSG_PEEK ), "Receiving" );
     355                ASSERTSOCKOP( ret = recv( fd, &bytes_received, sizeof( bytes_received ), MSG_PEEK ), "Receiving" );
    352356
    353357                if( ret == 0 )
     
    356360                if( ret < 4 )
    357361                {
    358                         imcb_log( df->ic, "WARNING: DCC SEND: receiver sent only 2 bytes instead of 4, shouldn't happen too often!" );
     362                        imcb_log( df->ic, "WARNING: DCC SEND: receiver sent only %d bytes instead of 4, shouldn't happen too often!", ret );
    359363                        return TRUE;
    360364                }
    361365
    362                 ASSERTSOCKOP( ret = recv( fd, &bytes_received, sizeof( bytes_received  ), 0 ), "Receiving" );
     366                ASSERTSOCKOP( ret = recv( fd, &bytes_received, sizeof( bytes_received ), 0 ), "Receiving" );
    363367                if( ret != 4 )
    364368                        return dcc_abort( df, "MSG_PEEK'ed 4, but can only dequeue %d bytes", ret );
     
    400404                return dcc_abort( df, "BUG: protocol didn't register write()" );
    401405       
    402         ASSERTSOCKOP( fd = df->fd = socket( saddr->ss_family, SOCK_STREAM, 0 ) , "Opening Socket" );
     406        ASSERTSOCKOP( fd = df->fd = socket( saddr->ss_family, SOCK_STREAM, 0 ), "Opening Socket" );
    403407
    404408        sock_make_nonblocking( fd );
     
    429433}
    430434
    431 gboolean dccs_recv_proto( gpointer data, gint fd, b_input_condition cond)
     435gboolean dccs_recv_proto( gpointer data, gint fd, b_input_condition cond )
    432436{
    433437        dcc_file_transfer_t *df = data;
  • protocols/jabber/Makefile

    r2288705 r1c3008a  
    1010
    1111# [SH] Program variables
    12 objects = conference.o io.o iq.o jabber.o jabber_util.o message.o presence.o sasl.o si.o s5bytestream.o
     12objects = conference.o io.o iq.o jabber.o jabber_util.o message.o presence.o s5bytestream.o sasl.o si.o
    1313
    1414CFLAGS += -Wall
  • protocols/jabber/iq.c

    r2288705 r1c3008a  
    9999                                                   XMLNS_MUC,
    100100                                                   XMLNS_PING,
    101                                                    XMLNS_SI,
    102                                                    XMLNS_BYTESTREAMS,
    103                                                    XMLNS_FILETRANSFER,
     101                                                   XMLNS_SI,
     102                                                   XMLNS_BYTESTREAMS,
     103                                                   XMLNS_FILETRANSFER,
    104104                                                   NULL };
    105105                        const char **f;
     
    127127        else if( strcmp( type, "set" ) == 0 )
    128128        {
    129                 if(  ( c = xt_find_node( node->children, "si" ) ) &&
    130                      ( strcmp( xt_find_attr( c, "xmlns" ), XMLNS_SI ) == 0 ) )
     129                if( ( c = xt_find_node( node->children, "si" ) ) &&
     130                    ( strcmp( xt_find_attr( c, "xmlns" ), XMLNS_SI ) == 0 ) )
    131131                {
    132132                        return jabber_si_handle_request( ic, node, c );
    133                 } else if( !( c = xt_find_node( node->children, "query" ) ) ||
    134                     !( s = xt_find_attr( c, "xmlns" ) ) )
     133                }
     134                else if( !( c = xt_find_node( node->children, "query" ) ) ||
     135                         !( s = xt_find_attr( c, "xmlns" ) ) )
    135136                {
    136137                        imcb_log( ic, "Warning: Received incomplete IQ-%s packet", type );
    137138                        return XT_HANDLED;
    138                 } else if( strcmp( s, XMLNS_ROSTER ) == 0 )
     139                }
     140                else if( strcmp( s, XMLNS_ROSTER ) == 0 )
    139141                {
    140142                /* This is a roster push. XMPP servers send this when someone
     
    160162                                pack = 0;
    161163                        }
    162                 } else if( strcmp( s, XMLNS_BYTESTREAMS ) == 0 )
    163                 {
    164                         /* Bytestream Request (stage 2 of file transfer) */
     164                }
     165                else if( strcmp( s, XMLNS_BYTESTREAMS ) == 0 )
     166                {
     167                        /* Bytestream Request (stage 2 of file transfer) */
    165168                        return jabber_bs_recv_request( ic, node, c );
    166                 } else
     169                }
     170                else
    167171                {
    168172                        xt_free_node( reply );
     
    655659       
    656660        c = c->children;
    657         while( ( c = xt_find_node( c, "feature" ) ) ) {
     661        while( ( c = xt_find_node( c, "feature" ) ) )
     662        {
    658663                feature = xt_find_attr( c, "var" );
    659                 bud->features = g_slist_append(bud->features, g_strdup(feature) );
     664                bud->features = g_slist_append( bud->features, g_strdup( feature ) );
    660665                c = c->next;
    661666        }
     
    719724                        c = c->next;
    720725                }
    721         } else if( strcmp( xt_find_attr( c, "xmlns" ), XMLNS_DISCO_INFO ) == 0 )
     726        }
     727        else if( strcmp( xt_find_attr( c, "xmlns" ), XMLNS_DISCO_INFO ) == 0 )
    722728        {
    723729                char *category, *type;
     
    737743                        c = c->next;
    738744                }
    739         } else if( strcmp( xt_find_attr( c, "xmlns" ), XMLNS_BYTESTREAMS ) == 0 )
     745        }
     746        else if( strcmp( xt_find_attr( c, "xmlns" ), XMLNS_BYTESTREAMS ) == 0 )
    740747        {
    741748                char *host, *jid;
     
    761768        if( jd->have_streamhosts == 0 )
    762769                jd->have_streamhosts++;
     770
    763771        return XT_HANDLED;
    764772}
  • protocols/jabber/jabber.c

    r2288705 r1c3008a  
    6363       
    6464        s = set_add( &acc->set, "priority", "0", set_eval_priority, acc );
     65
     66        s = set_add( &acc->set, "proxy", "<local>;<auto>", NULL, acc );
    6567       
    6668        s = set_add( &acc->set, "resource", "BitlBee", NULL, acc );
     
    8082        s = set_add( &acc->set, "xmlconsole", "false", set_eval_bool, acc );
    8183        s->flags |= ACC_SET_OFFLINE_ONLY;
    82 
    83         s = set_add( &acc->set, "proxy", "<local>;<auto>", NULL, acc );
    8484}
    8585
  • protocols/jabber/jabber.h

    r2288705 r1c3008a  
    6060} jabber_buddy_flags_t;
    6161
    62 /* Stores a streamhost's(a.k.a. proxy) data */
     62/* Stores a streamhost's (a.k.a. proxy) data */
    6363typedef struct
    6464{
  • protocols/msn/Makefile

    r2288705 r1c3008a  
    1010
    1111# [SH] Program variables
    12 objects = msn.o msn_util.o ns.o passport.o sb.o tables.o invitation.o
     12objects = invitation.o msn.o msn_util.o ns.o passport.o sb.o tables.o
    1313
    1414CFLAGS += -Wall
  • protocols/msn/msn.c

    r2288705 r1c3008a  
    7878        {
    7979                while( md->filetransfers ) {
    80                         imcb_file_canceled( md->filetransfers->data, "Closing msn connection" );
     80                        imcb_file_canceled( md->filetransfers->data, "Closing connection" );
    8181                }
    8282               
  • protocols/msn/sb.c

    r2288705 r1c3008a  
    174174                        buf = g_new0( char, i );
    175175                        i = g_snprintf( buf, i, MSN_TYPING_HEADERS, sb->ic->acc->user );
    176                 } else if( strncmp( text, MSN_INVITE_HEADERS, sizeof( MSN_INVITE_HEADERS ) - 1 ) == 0 )
     176                }
     177                else if( strncmp( text, MSN_INVITE_HEADERS, sizeof( MSN_INVITE_HEADERS ) - 1 ) == 0 )
    177178                {
    178179                        buf = g_strdup( text );
    179180                        i = strlen( buf );
    180                 } else
     181                }
     182                else
    181183                {
    182184                        buf = g_new0( char, sizeof( MSN_MESSAGE_HEADERS ) + strlen( text ) * 2 + 1 );
Note: See TracChangeset for help on using the changeset viewer.