Changeset 6b13103 for protocols


Ignore:
Timestamp:
2015-01-16T19:50:23Z (5 years ago)
Author:
dequis <dx@…>
Branches:
master
Children:
1065dd4, eabe6d4
Parents:
6f10697
Message:

Replace isdigit/isalpha/.../tolower/toupper with glib variants

This fixes warnings about passing signed chars to them (apparently they
are implemented as macros that do array lookups without checks in some
platforms, yay)

Specifically:

functions=isalnum|isalpha|isdigit|isspace|isxdigit|tolower|toupper
sed -ir "s/$functions/g_ascii_&/g" /*.c

Location:
protocols
Files:
13 edited

Legend:

Unmodified
Added
Removed
  • protocols/account.c

    r6f10697 r6b13103  
    8181        if( strcmp( prpl->name, "oscar" ) == 0 )
    8282        {
    83                 if( isdigit( a->user[0] ) )
     83                if( g_ascii_isdigit( a->user[0] ) )
    8484                        strcpy( tag, "icq" );
    8585                else
     
    417417       
    418418        /* Format: /[0-9]+([*+][0-9]+(<[0-9+])?)?/ */
    419         while( *value && isdigit( *value ) )
     419        while( *value && g_ascii_isdigit( *value ) )
    420420                p->start = p->start * 10 + *value++ - '0';
    421421       
     
    433433       
    434434        /* + or * the delay by this number every time. */
    435         while( *value && isdigit( *value ) )
     435        while( *value && g_ascii_isdigit( *value ) )
    436436                p->step = p->step * 10 + *value++ - '0';
    437437       
     
    444444        p->max = 0;
    445445        value ++;
    446         while( *value && isdigit( *value ) )
     446        while( *value && g_ascii_isdigit( *value ) )
    447447                p->max = p->max * 10 + *value++ - '0';
    448448       
  • protocols/jabber/jabber_util.c

    r6f10697 r6b13103  
    321321                        return FALSE;
    322322                }
    323                 if( tolower( jid1[i] ) != tolower( jid2[i] ) )
     323                if( g_ascii_tolower( jid1[i] ) != g_ascii_tolower( jid2[i] ) )
    324324                {
    325325                        return FALSE;
     
    342342           it's Unicode but feck Unicode. :-P So stop once we see a slash. */
    343343        for( i = 0; i < len && orig[i] != '/' ; i ++ )
    344                 new[i] = tolower( orig[i] );
     344                new[i] = g_ascii_tolower( orig[i] );
    345345        for( ; orig[i]; i ++ )
    346346                new[i] = orig[i];
  • protocols/jabber/sasl.c

    r6f10697 r6b13103  
    187187        len = strlen( field );
    188188       
    189         while( isspace( *data ) || *data == ',' )
     189        while( g_ascii_isspace( *data ) || *data == ',' )
    190190                data ++;
    191191       
     
    210210                        if( data[i] == ',' )
    211211                        {
    212                                 while( isspace( data[i] ) || data[i] == ',' )
     212                                while( g_ascii_isspace( data[i] ) || data[i] == ',' )
    213213                                        i ++;
    214214                               
  • protocols/msn/msn.c

    r6f10697 r6b13103  
    377377        g_tree_insert( md->domaintree, bu->handle, bu );
    378378       
    379         for( handle = bu->handle; isdigit( *handle ); handle ++ );
     379        for( handle = bu->handle; g_ascii_isdigit( *handle ); handle ++ );
    380380        if( *handle == ':' )
    381381        {
  • protocols/msn/ns.c

    r6f10697 r6b13103  
    581581                ic->flags |= OPT_PONGED;
    582582        }
    583         else if( isdigit( cmd[0][0] ) )
     583        else if( g_ascii_isdigit( cmd[0][0] ) )
    584584        {
    585585                int num = atoi( cmd[0] );
     
    997997        /* This might be a federated contact. Get its network number,
    998998           prefixed to bu->handle with a colon. Default is 1. */
    999         for( handle = bu->handle; isdigit( *handle ); handle ++ )
     999        for( handle = bu->handle; g_ascii_isdigit( *handle ); handle ++ )
    10001000                type = type * 10 + *handle - '0';
    10011001        if( *handle == ':' )
  • protocols/msn/sb.c

    r6f10697 r6b13103  
    506506        else if( strcmp( cmd[0], "CAL" ) == 0 )
    507507        {
    508                 if( num_parts < 4 || !isdigit( cmd[3][0] ) )
     508                if( num_parts < 4 || !g_ascii_isdigit( cmd[3][0] ) )
    509509                {
    510510                        msn_sb_destroy( sb );
     
    645645                }
    646646        }
    647         else if( isdigit( cmd[0][0] ) )
     647        else if( g_ascii_isdigit( cmd[0][0] ) )
    648648        {
    649649                int num = atoi( cmd[0] );
  • protocols/oscar/auth.c

    r6f10697 r6b13103  
    125125                return -EINVAL;
    126126
    127         if (isdigit(sn[0]) && set_getbool(&ic->acc->set, "old_icq_auth"))
     127        if (g_ascii_isdigit(sn[0]) && set_getbool(&ic->acc->set, "old_icq_auth"))
    128128                return goddamnicq(sess, conn, sn);
    129129
  • protocols/oscar/oscar.c

    r6f10697 r6b13103  
    368368{
    369369        set_t *s;
    370         gboolean icq = isdigit(acc->user[0]);
     370        gboolean icq = g_ascii_isdigit(acc->user[0]);
    371371       
    372372        if (icq) {
     
    394394        struct oscar_data *odata = ic->proto_data = g_new0(struct oscar_data, 1);
    395395
    396         if (isdigit(acc->user[0]))
     396        if (g_ascii_isdigit(acc->user[0]))
    397397                odata->icq = TRUE;
    398398        else
     
    25002500        char * chatname, *s;
    25012501       
    2502         chatname = g_strdup_printf("%s%s%d", isdigit(*ic->acc->user) ? "icq" : "",
     2502        chatname = g_strdup_printf("%s%s%d", g_ascii_isdigit(*ic->acc->user) ? "icq" : "",
    25032503                                   ic->acc->user, chat_id++);
    25042504       
    25052505        for (s = chatname; *s; s ++)
    2506                 if (!isalnum(*s))
     2506                if (!g_ascii_isalnum(*s))
    25072507                        *s = '0';
    25082508       
  • protocols/oscar/oscar_util.c

    r6f10697 r6b13103  
    5757                                curPtr2++;
    5858                } else {
    59                         if ( toupper(*curPtr1) != toupper(*curPtr2))
     59                        if ( g_ascii_toupper(*curPtr1) != g_ascii_toupper(*curPtr2))
    6060                                return 1;
    6161                        curPtr1++;
  • protocols/purple/purple.c

    r6f10697 r6b13103  
    6161                while( *a == '_' ) a ++;
    6262                while( *b == '_' ) b ++;
    63                 if( tolower( *a ) != tolower( *b ) )
     63                if( g_ascii_tolower( *a ) != g_ascii_tolower( *b ) )
    6464                        return FALSE;
    6565               
     
    11481148                                {
    11491149                                        n = strlen( value ) - 1;
    1150                                         while( isspace( value[n] ) )
     1150                                        while( g_ascii_isspace( value[n] ) )
    11511151                                                n --;
    11521152                                        g_string_append_len( info, value, n + 1 );
  • protocols/skype/skype.c

    r6f10697 r6b13103  
    514514                                char *iptr = sd->info_sex;
    515515                                while (*iptr++)
    516                                         *iptr = tolower(*iptr);
     516                                        *iptr = g_ascii_tolower(*iptr);
    517517                                g_string_append_printf(st,
    518518                                        "Gender: %s\n", sd->info_sex);
  • protocols/twitter/twitter.c

    r6f10697 r6b13103  
    459459
    460460                        strcpy(pin, message);
    461                         for (s = pin + sizeof(pin) - 2; s > pin && isspace(*s); s--)
     461                        for (s = pin + sizeof(pin) - 2; s > pin && g_ascii_isspace(*s); s--)
    462462                                *s = '\0';
    463                         for (s = pin; *s && isspace(*s); s++) {
     463                        for (s = pin; *s && g_ascii_isspace(*s); s++) {
    464464                        }
    465465
  • protocols/yahoo/yahoo_httplib.c

    r6f10697 r6b13103  
    152152static int isurlchar(unsigned char c)
    153153{
    154         return (isalnum(c));
     154        return (g_ascii_isalnum(c));
    155155}
    156156
Note: See TracChangeset for help on using the changeset viewer.