Changeset 9464


Ignore:
Timestamp:
Nov 1, 2009, 1:35:06 AM (13 years ago)
Author:
charles
Message:

(trunk libT) use attribute((nonnull)) where appropriate. remove two unused functions tr_int2ptr(), tr_ptr2int()

Location:
trunk/libtransmission
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/libtransmission/request-list.c

    r8889 r9464  
    7676
    7777void
    78 reqListAppend( struct request_list *       list,
    79                const struct peer_request * req )
     78reqListAppend( struct request_list * list, const struct peer_request * req )
    8079{
    8180    int low;
     81    tr_bool exact;
    8282
    8383    reqListReserve( list, list->len + 8 );
     
    8787
    8888    /* insert into list->sort */
    89     low = reqListSortPos( list, req, NULL );
     89    low = reqListSortPos( list, req, &exact );
    9090    memmove( &list->sort[low+1], &list->sort[low], (list->len-low)*sizeof(struct peer_request) );
    9191    list->sort[low] = *req;
  • trunk/libtransmission/utils.c

    r9463 r9464  
    10831083}
    10841084
    1085 int
    1086 tr_ptr2int( void* v )
    1087 {
    1088     return (intptr_t)v;
    1089 }
    1090 
    1091 void*
    1092 tr_int2ptr( int i )
    1093 {
    1094     return (void*)(intptr_t)i;
    1095 }
    1096 
    10971085/***
    10981086****
     
    11101098    const char * cbase = base;
    11111099    tr_bool exact = FALSE;
    1112     int c;
    1113 
    1114     while( nmemb )
     1100
     1101    while( nmemb != 0 )
    11151102    {
    11161103        const size_t half = nmemb / 2;
    11171104        const size_t middle = first + half;
    1118         c = compar( key, cbase + size*middle );
     1105        const int c = compar( key, cbase + size*middle );
    11191106
    11201107        if( c <= 0 ) {
     
    11281115    }
    11291116
    1130     if( exact_match )
    1131         *exact_match = exact;
     1117    *exact_match = exact;
    11321118
    11331119    return first;
  • trunk/libtransmission/utils.h

    r9421 r9464  
    5555#ifndef TR_GNUC_PRINTF
    5656 #ifdef __GNUC__
    57   #define TR_GNUC_PRINTF( fmt,\
    58                           args ) __attribute__ ( ( format ( printf, fmt,\
    59                                                             args ) ) )
     57  #define TR_GNUC_PRINTF( fmt, args ) __attribute__ ( ( format ( printf, fmt, args ) ) )
    6058 #else
    6159  #define TR_GNUC_PRINTF( fmt, args )
     
    6361#endif
    6462
     63#ifndef TR_GNUC_NONNULL
     64 #ifdef __GNUC__
     65  #define TR_GNUC_NONNULL( ... ) __attribute__((nonnull (__VA_ARGS__)))
     66 #else
     67  #define TR_GNUC_NONNULL( ... )
     68 #endif
     69#endif
     70
    6571#ifndef TR_GNUC_NULL_TERMINATED
    6672 #if __GNUC__ >= 4
    6773  #define TR_GNUC_NULL_TERMINATED __attribute__ ( ( __sentinel__ ) )
     74  #define TR_GNUC_HOT __attribute ( ( hot ) )
    6875 #else
    6976  #define TR_GNUC_NULL_TERMINATED
     77  #define TR_GNUC_HOT
    7078 #endif
    7179#endif
     
    189197                           const char * name,
    190198                           const char * fmt,
    191                            ... ) TR_GNUC_PRINTF( 4, 5 );
     199                           ... ) TR_GNUC_PRINTF( 4, 5 ) TR_GNUC_NONNULL(1,4);
    192200
    193201char*          tr_getLogTimeStr( char * buf,
    194                                  int    buflen );
     202                                 int    buflen ) TR_GNUC_NONNULL(1);
    195203
    196204
    197205int            tr_wildmat( const char * text,
    198                            const char * pattern );
     206                           const char * pattern ) TR_GNUC_NONNULL(1,2);
    199207
    200208/** @brief Portability wrapper for basename() that uses the system implementation if available */
     
    214222 */
    215223int            tr_mkdir( const char * path,
    216                          int          permissions );
     224                         int          permissions ) TR_GNUC_NONNULL(1);
    217225
    218226/**
     
    222230 * (in which case errno is set appropriately).
    223231 */
    224 int tr_mkdirp( const char * path, int permissions );
     232int tr_mkdirp( const char * path, int permissions ) TR_GNUC_NONNULL(1);
    225233
    226234
     
    229237 * On failure, NULL is returned and errno is set.
    230238 */
    231 uint8_t* tr_loadFile( const char * filename, size_t * size ) TR_GNUC_MALLOC;
     239uint8_t* tr_loadFile( const char * filename, size_t * size ) TR_GNUC_MALLOC TR_GNUC_NONNULL(1);
    232240
    233241
     
    324332                   size_t       size,
    325333                   int       (* compar)(const void* key, const void* arrayMember),
    326                    tr_bool    * exact_match );
     334                   tr_bool    * exact_match ) TR_GNUC_HOT TR_GNUC_NONNULL(1,5,6);
    327335
    328336
     
    343351/** @brief Portability wrapper for snprintf() that uses the system implementation if available */
    344352int tr_snprintf( char * buf, size_t buflen,
    345                  const char * fmt, ... ) TR_GNUC_PRINTF( 3, 4 );
     353                 const char * fmt, ... ) TR_GNUC_PRINTF( 3, 4 ) TR_GNUC_NONNULL(1,3);
    346354
    347355const char* tr_strerror( int );
     
    361369typedef void ( tr_set_func )( void * element, void * userData );
    362370
    363 void        tr_set_compare( const void * a,
    364                             size_t aCount,
    365                             const void * b,
    366                             size_t bCount,
     371void        tr_set_compare( const void * a, size_t aCount,
     372                            const void * b, size_t bCount,
    367373                            int compare( const void * a, const void * b ),
    368374                            size_t elementSize,
     
    373379
    374380void tr_sha1_to_hex( char *          out,
    375                      const uint8_t * sha1 );
    376 
    377 
    378 tr_bool tr_httpIsValidURL( const char * url );
     381                     const uint8_t * sha1 ) TR_GNUC_NONNULL(1,2);
     382
     383
     384tr_bool tr_httpIsValidURL( const char * url ) TR_GNUC_NONNULL(1);
    379385
    380386int  tr_httpParseURL( const char * url,
     
    382388                      char **      setme_host,
    383389                      int *        setme_port,
    384                       char **      setme_path );
     390                      char **      setme_path ) TR_GNUC_NONNULL(1);
    385391
    386392double tr_getRatio( double numerator, double denominator );
    387393
    388 int* tr_parseNumberRange( const char * str, int str_len, int * setmeCount ) TR_GNUC_MALLOC;
    389 
    390 
    391 int tr_ptr2int( void* );
    392 
    393 void* tr_int2ptr( int );
     394int* tr_parseNumberRange( const char * str, int str_len, int * setmeCount ) TR_GNUC_MALLOC TR_GNUC_NONNULL(1);
     395
    394396
    395397/* truncate a double value at a given number of decimal places.
     
    413415 * @param the string represntation of "infinity"
    414416 */
    415 char* tr_strratio( char * buf, size_t buflen, double ratio, const char * infinity );
     417char* tr_strratio( char * buf, size_t buflen, double ratio, const char * infinity ) TR_GNUC_NONNULL(1,4);
    416418
    417419/** @brief Portability wrapper for localtime_r() that uses the system implementation if available */
     
    420422
    421423/** on success, return 0.  on failure, return -1 and set errno */
    422 int tr_moveFile( const char * oldpath, const char * newpath, tr_bool * renamed );
     424int tr_moveFile( const char * oldpath, const char * newpath, tr_bool * renamed ) TR_GNUC_NONNULL(1,2);
    423425
    424426
Note: See TracChangeset for help on using the changeset viewer.