Changeset 5645


Ignore:
Timestamp:
Apr 18, 2008, 11:17:40 PM (13 years ago)
Author:
charles
Message:

minor code cleanup: remove dead code, const-correctness, etc

Location:
trunk/libtransmission
Files:
5 edited

Legend:

Unmodified
Added
Removed
  • trunk/libtransmission/inout.c

    r5644 r5645  
    6969    if ((ioMode==TR_IO_READ) && !fileExists ) /* does file exist? */
    7070        err = tr_ioErrorFromErrno( errno );
    71     else if ((fd = tr_fdFileCheckout ( tor->destination, file->name, ioMode==TR_IO_WRITE )) < 0)
     71    else if ((fd = tr_fdFileCheckout ( tor->destination,
     72                                       file->name,
     73                                       ioMode==TR_IO_WRITE )) < 0)
    7274        err = fd;
    7375    else if( lseek( fd, (off_t)fileOffset, SEEK_SET ) == ((off_t)-1) )
     
    8789}
    8890
    89 static tr_errno
     91static void
    9092findFileLocation( const tr_torrent * tor,
    9193                  tr_piece_index_t   pieceIndex,
     
    110112        size_t half = len / 2;
    111113        size_t middle = first + half;
    112         if( inf->files[middle].offset + inf->files[middle].length < offset ) {
     114        if( inf->files[middle].offset + inf->files[middle].length < offset )
     115        {
    113116            first = middle;
    114117            ++first;
     
    125128    assert( *fileIndex < inf->fileCount );
    126129    assert( *fileOffset < inf->files[first].length );
    127     return 0;
    128130}
    129131
     
    162164
    163165static tr_errno
    164 readOrWritePiece( tr_torrent        * tor,
    165                   int                 ioMode,
    166                   tr_piece_index_t    pieceIndex,
    167                   uint32_t            pieceOffset,
    168                   uint8_t           * buf,
    169                   size_t              buflen )
     166readOrWritePiece( const tr_torrent        * tor,
     167                  int                       ioMode,
     168                  tr_piece_index_t          pieceIndex,
     169                  uint32_t                  pieceOffset,
     170                  uint8_t                 * buf,
     171                  size_t                    buflen )
    170172{
    171173    tr_errno err = 0;
     
    174176    const tr_info * info = &tor->info;
    175177
    176     assert( pieceIndex < tor->info.pieceCount );
    177     assert( pieceOffset + buflen <= tr_torPieceCountBytes( tor, pieceIndex ) );
    178 
    179     if( !err )
    180         err = findFileLocation ( tor, pieceIndex, pieceOffset, &fileIndex, &fileOffset );
     178    if( pieceIndex >= tor->info.pieceCount )
     179        return TR_ERROR_ASSERT;
     180    if( pieceOffset + buflen > tr_torPieceCountBytes( tor, pieceIndex ) )
     181        return TR_ERROR_ASSERT;
     182
     183    findFileLocation ( tor, pieceIndex, pieceOffset,
     184                            &fileIndex, &fileOffset );
    181185
    182186    while( buflen && !err )
     
    195199        buf += bytesThisPass;
    196200        buflen -= bytesThisPass;
    197         fileIndex++;
     201        ++fileIndex;
    198202        fileOffset = 0;
    199203    }
     
    209213           uint8_t           * buf )
    210214{
    211     return readOrWritePiece( (tr_torrent*)tor, TR_IO_READ, pieceIndex, begin, buf, len );
     215    return readOrWritePiece( tor, TR_IO_READ, pieceIndex, begin, buf, len );
    212216}
    213217
    214218tr_errno
    215 tr_ioWrite( tr_torrent        * tor,
     219tr_ioWrite( const tr_torrent  * tor,
    216220            tr_piece_index_t    pieceIndex,
    217221            uint32_t            begin,
     
    227231
    228232static tr_errno
    229 tr_ioRecalculateHash( const tr_torrent  * tor,
    230                       tr_piece_index_t    pieceIndex,
    231                       uint8_t           * setme )
     233recalculateHash( const tr_torrent  * tor,
     234                 tr_piece_index_t    pieceIndex,
     235                 uint8_t           * setme )
    232236{
    233237    static uint8_t * buf = NULL;
     
    272276    uint8_t hash[SHA_DIGEST_LENGTH];
    273277
    274     err  = tr_ioRecalculateHash( tor, pieceIndex, hash );
    275 
    276     if( !err && memcmp( hash, tor->info.pieces[pieceIndex].hash, SHA_DIGEST_LENGTH ) )
     278    err  = recalculateHash( tor, pieceIndex, hash );
     279
     280    if( !err && memcmp( hash, tor->info.pieces[pieceIndex].hash,
     281                        SHA_DIGEST_LENGTH ) )
    277282        err = TR_ERROR_IO_CHECKSUM;
    278283
  • trunk/libtransmission/inout.h

    r5329 r5645  
    3333 * or TR_ERROR_IO_* otherwise.
    3434 */
    35 int tr_ioRead  ( const struct tr_torrent  * tor,
    36                  tr_piece_index_t           pieceIndex,
    37                  uint32_t                   offset,
    38                  uint32_t                   len,
    39                  uint8_t                  * setme );
     35tr_errno tr_ioRead( const struct tr_torrent  * tor,
     36                    tr_piece_index_t           pieceIndex,
     37                    uint32_t                   offset,
     38                    uint32_t                   len,
     39                    uint8_t                  * setme );
    4040
    4141/**
     
    4444 * or TR_ERROR_IO_* otherwise.
    4545 */
    46 tr_errno tr_ioWrite ( struct tr_torrent  * tor,
    47                       tr_piece_index_t     pieceIndex,
    48                       uint32_t             offset,
    49                       uint32_t             len,
    50                       const uint8_t      * writeme );
     46tr_errno tr_ioWrite( const struct tr_torrent  * tor,
     47                     tr_piece_index_t           pieceIndex,
     48                     uint32_t                   offset,
     49                     uint32_t                   len,
     50                     const uint8_t            * writeme );
    5151
    5252/**
  • trunk/libtransmission/platform.c

    r5642 r5645  
    162162}
    163163   
    164 void
    165 tr_threadJoin( tr_thread * t )
    166 {
    167     if( t != NULL )
    168     {
    169 #ifdef __BEOS__
    170         long exit;
    171         wait_for_thread( t->thread, &exit );
    172 #elif defined(WIN32)
    173         WaitForSingleObject( t->thread_handle, INFINITE );
    174         CloseHandle( t->thread_handle );
    175 #else
    176         pthread_join( t->thread, NULL );
    177 #endif
    178 
    179         tr_dbg( "Thread '%s' joined", t->name );
    180         t->name = NULL;
    181         t->func = NULL;
    182         tr_free( t );
    183     }
    184 }
    185 
    186164/***
    187165****  LOCKS
     
    211189    l->lock = create_sem( 1, "" );
    212190#elif defined(WIN32)
    213     InitializeCriticalSection( &l->lock ); /* critical sections support recursion */
     191    InitializeCriticalSection( &l->lock ); /* supports recursion */
    214192#else
    215193    pthread_mutexattr_t attr;
     
    301279#else
    302280            struct passwd * pw = getpwuid( getuid() );
    303             endpwent( );
    304281            if( pw )
    305282                home = tr_strdup( pw->pw_dir );
     283            endpwent( );
    306284#endif
    307285        }
  • trunk/libtransmission/platform.h

    r5517 r5645  
    4646
    4747tr_thread*   tr_threadNew  ( void (*func)(void *), void * arg, const char * name );
    48 void         tr_threadJoin ( tr_thread * );
    4948int          tr_amInThread ( const tr_thread * );
    5049
  • trunk/libtransmission/resume.c

    r5643 r5645  
    6767                                          tor->info.hash, &pex );
    6868    if( count > 0 )
    69         tr_bencInitRaw( tr_bencDictAdd( dict, KEY_PEERS ), pex, sizeof(tr_pex)*count );
     69        tr_bencDictAddRaw( dict, KEY_PEERS, pex, sizeof(tr_pex)*count );
    7070    tr_free( pex );
    7171}
     
    250250    tr_benc * p;
    251251    tr_benc * m;
    252     tr_benc * b;
    253252    const tr_bitfield * bitfield;
    254253
     
    267266    /* add the bitfield */
    268267    bitfield = tr_cpBlockBitfield( tor->completion );
    269     b = tr_bencDictAdd( p, KEY_PROGRESS_BITFIELD );
    270     tr_bencInitRaw( b, bitfield->bits, bitfield->len );
     268    tr_bencDictAddRaw( p, KEY_PROGRESS_BITFIELD,
     269                      bitfield->bits, bitfield->len );
    271270
    272271    /* cleanup */
Note: See TracChangeset for help on using the changeset viewer.