Changeset 6b13103


Ignore:
Timestamp:
2015-01-16T19:50:23Z (9 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

Files:
26 edited

Legend:

Unmodified
Added
Removed

  • dcc.c

    r6f10697 r6b13103  
    523523               
    524524                host = ctcp[3];
    525                 while( *host && isdigit( *host ) ) host++; /* Just digits? */
     525                while( *host && g_ascii_isdigit( *host ) ) host++; /* Just digits? */
    526526                if( *host == '\0' )
    527527                {

  • irc_channel.c

    r6f10697 r6b13103  
    575575       
    576576        /* Scan for non-whitespace chars followed by a colon: */
    577         for( s = msg; *s && !isspace( *s ) && *s != ':' && *s != ','; s ++ ) {}
     577        for( s = msg; *s && !g_ascii_isspace( *s ) && *s != ':' && *s != ','; s ++ ) {}
    578578       
    579579        if( *s == ':' || *s == ',' )
    … …  
    583583                memset( to, 0, sizeof( to ) );
    584584                strncpy( to, msg, s - msg );
    585                 while( *(++s) && isspace( *s ) ) {}
     585                while( *(++s) && g_ascii_isspace( *s ) ) {}
    586586                msg = s;
    587587               

  • irc_im.c

    r6f10697 r6b13103  
    7373                {
    7474                        char *s;
    75                         for( s = bu->ic->acc->tag; isalnum( *s ); s ++ );
     75                        for( s = bu->ic->acc->tag; g_ascii_isalnum( *s ); s ++ );
    7676                        /* Only use the tag if we got to the end of the string.
    7777                           (So allow alphanumerics only. Hopefully not too
    … …  
    316316           TODO(wilmer): Do the same with away msgs again! */
    317317        for( s = iu->fullname; *s; s ++ )
    318                 if( isspace( *s ) ) *s = ' ';
     318                if( g_ascii_isspace( *s ) ) *s = ' ';
    319319       
    320320        if( ( bu->ic->flags & OPT_LOGGED_IN ) && set_getbool( &bee->set, "display_namechanges" ) )

  • irc_util.c

    r6f10697 r6b13103  
    4242       
    4343        /* \d+ */
    44         if( !isdigit( *s ) )
     44        if( !g_ascii_isdigit( *s ) )
    4545                return SET_INVALID;
    46         while( *s && isdigit( *s ) ) s ++;
     46        while( *s && g_ascii_isdigit( *s ) ) s ++;
    4747       
    4848        /* EOS? */
    … …  
    5656       
    5757        /* \d+ */
    58         if( !isdigit( *s ) )
     58        if( !g_ascii_isdigit( *s ) )
    5959                return SET_INVALID;
    60         while( *s && isdigit( *s ) ) s ++;
     60        while( *s && g_ascii_isdigit( *s ) ) s ++;
    6161       
    6262        /* EOS */

  • lib/http_client.c

    r6f10697 r6b13103  
    345345                /* Might be a \r\n from the last chunk. */
    346346                s = chunk;
    347                 while( isspace( *s ) )
     347                while( g_ascii_isspace( *s ) )
    348348                        s ++;
    349349                /* Chunk length. Might be incomplete. */
    350350                if( s < eos && sscanf( s, "%x", &clen ) != 1 )
    351351                        return CR_ERROR;
    352                 while( isxdigit( *s ) )
     352                while( g_ascii_isxdigit( *s ) )
    353353                        s ++;
    354354               

  • lib/ini.c

    r6f10697 r6b13103  
    6363        char *e;
    6464
    65         while( isspace( *in ) )
     65        while( g_ascii_isspace( *in ) )
    6666                in++;
    6767
    6868        e = in + strlen( in ) - 1;
    69         while( e > in && isspace( *e ) )
     69        while( e > in && g_ascii_isspace( *e ) )
    7070                e--;
    7171        e[1] = 0;

  • lib/json.c

    r6f10697 r6b13103  
    5353static unsigned char hex_value (json_char c)
    5454{
    55    if (isdigit(c))
     55   if (g_ascii_isdigit(c))
    5656      return c - '0';
    5757
    … …  
    609609                     default:
    610610
    611                         if (isdigit (b) || b == '-')
     611                        if (g_ascii_isdigit (b) || b == '-')
    612612                        {
    613613                           if (!new_value (&state, &top, &root, &alloc, json_integer))
    … …  
    616616                           if (!state.first_pass)
    617617                           {
    618                               while (isdigit (b) || b == '+' || b == '-'
     618                              while (g_ascii_isdigit (b) || b == '+' || b == '-'
    619619                                        || b == 'e' || b == 'E' || b == '.')
    620620                              {
    … …  
    706706            case json_double:
    707707
    708                if (isdigit (b))
     708               if (g_ascii_isdigit (b))
    709709               {
    710710                  ++ num_digits;

  • lib/misc.c

    r6f10697 r6b13103  
    163163        while( *in )
    164164        {
    165                 if( *in == '<' && ( isalpha( *(in+1) ) || *(in+1) == '/' ) )
     165                if( *in == '<' && ( g_ascii_isalpha( *(in+1) ) || *(in+1) == '/' ) )
    166166                {
    167167                        /* If in points at a < and in+1 points at a letter or a slash, this is probably
    … …  
    198198                {
    199199                        cs = ++in;
    200                         while( *in && isalpha( *in ) )
     200                        while( *in && g_ascii_isalpha( *in ) )
    201201                                in ++;
    202202                       
    … …  
    314314        for( i = j = 0; t[i]; i ++, j ++ )
    315315        {
    316                 /* Warning: isalnum() is locale-aware, so don't use it here! */
     316                /* Warning: g_ascii_isalnum() is locale-aware, so don't use it here! */
    317317                if( ( t[i] >= 'A' && t[i] <= 'Z' ) ||
    318318                    ( t[i] >= 'a' && t[i] <= 'z' ) ||
    … …  
    490490       
    491491        while( *value )
    492                 if( !isdigit( *value ) )
     492                if( !g_ascii_isdigit( *value ) )
    493493                        return 0;
    494494                else

  • lib/ssl_gnutls.c

    r6f10697 r6b13103  
    318318        gnutls_set_default_priority( conn->session );
    319319        gnutls_credentials_set( conn->session, GNUTLS_CRD_CERTIFICATE, xcred );
    320         if( conn->hostname && !isdigit( conn->hostname[0] ) )
     320        if( conn->hostname && !g_ascii_isdigit( conn->hostname[0] ) )
    321321                gnutls_server_name_set( conn->session, GNUTLS_NAME_DNS,
    322322                                        conn->hostname, strlen( conn->hostname ) );

  • lib/ssl_openssl.c

    r6f10697 r6b13103  
    159159        SSL_set_fd( conn->ssl, conn->fd );
    160160       
    161         if( conn->hostname && !isdigit( conn->hostname[0] ) )
     161        if( conn->hostname && !g_ascii_isdigit( conn->hostname[0] ) )
    162162                SSL_set_tlsext_host_name( conn->ssl, conn->hostname );
    163163       

  • nick.c

    r6f10697 r6b13103  
    4141        do {
    4242                if (*orig != ' ')
    43                         new[i++] = tolower( *orig );
     43                        new[i++] = g_ascii_tolower( *orig );
    4444        }
    4545        while (*(orig++));
    … …  
    144144                                fmt += 2;
    145145                        }
    146                         else if( isdigit( *fmt ) )
     146                        else if( g_ascii_isdigit( *fmt ) )
    147147                        {
    148148                                len = 0;
    149149                                /* Grab a number. */
    150                                 while( isdigit( *fmt ) )
     150                                while( g_ascii_isdigit( *fmt ) )
    151151                                        len = len * 10 + ( *(fmt++) - '0' );
    152152                        }
    … …  
    331331                }
    332332        }
    333         if( isdigit( nick[0] ) )
     333        if( g_ascii_isdigit( nick[0] ) )
    334334        {
    335335                char *orig;
    … …  
    351351       
    352352        /* Empty/long nicks are not allowed, nor numbers at [0] */
    353         if( !*nick || isdigit( nick[0] ) || strlen( nick ) > MAX_NICK_LENGTH )
     353        if( !*nick || g_ascii_isdigit( nick[0] ) || strlen( nick ) > MAX_NICK_LENGTH )
    354354                return 0;
    355355       

  • otr.c

    r6f10697 r6b13103  
    14701470int hexval(char a)
    14711471{
    1472         int x=tolower(a);
     1472        int x=g_ascii_tolower(a);
    14731473       
    14741474        if(x>='a' && x<='f')
    … …  
    15571557        for(i=0; args[i]; i++) {
    15581558                for(j=0; args[i][j]; j++) {
    1559                         char c = toupper(args[i][j]);
     1559                        char c = g_ascii_toupper(args[i][j]);
    15601560                       
    15611561                        if(n>=40) {
    … …  
    16211621        for(i=0; args[i]; i++) {
    16221622                for(j=0; args[i][j]; j++) {
    1623                         char c = toupper(args[i][j]);
     1623                        char c = g_ascii_toupper(args[i][j]);
    16241624                       
    16251625                        if(n>=40) {

  • 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

  • set.c

    r6f10697 r6b13103  
    226226       
    227227        for( ; *s; s ++ )
    228                 if( !isdigit( *s ) )
     228                if( !g_ascii_isdigit( *s ) )
    229229                        return SET_INVALID;
    230230       
Note: See TracChangeset for help on using the changeset viewer.