Changeset 9669


Ignore:
Timestamp:
Dec 5, 2009, 1:02:37 AM (12 years ago)
Author:
livings124
Message:

remove a redundant level check in tr_msg()

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/libtransmission/utils.c

    r9647 r9669  
    299299{
    300300    const int err = errno; /* message logging shouldn't affect errno */
     301    char buf[MAX_STACK_ARRAY_SIZE];
     302    va_list ap;
     303
    301304    tr_lockLock( messageLock );
    302305
    303     if( messageLevel >= level )
    304     {
    305         char buf[MAX_STACK_ARRAY_SIZE];
    306         va_list ap;
    307 
    308         /* build the text message */
    309         *buf = '\0';
    310         va_start( ap, fmt );
    311         evutil_vsnprintf( buf, sizeof( buf ), fmt, ap );
    312         va_end( ap );
    313 
    314         OutputDebugString( buf );
    315 
    316         if( *buf )
     306    /* build the text message */
     307    *buf = '\0';
     308    va_start( ap, fmt );
     309    evutil_vsnprintf( buf, sizeof( buf ), fmt, ap );
     310    va_end( ap );
     311
     312    OutputDebugString( buf );
     313
     314    if( *buf )
     315    {
     316        if( messageQueuing )
    317317        {
    318             if( messageQueuing )
     318            tr_msg_list * newmsg;
     319            newmsg = tr_new0( tr_msg_list, 1 );
     320            newmsg->level = level;
     321            newmsg->when = tr_time( );
     322            newmsg->message = tr_strdup( buf );
     323            newmsg->file = file;
     324            newmsg->line = line;
     325            newmsg->name = tr_strdup( name );
     326
     327            *messageQueueTail = newmsg;
     328            messageQueueTail = &newmsg->next;
     329            ++messageQueueCount;
     330           
     331            if (messageQueueCount > TR_MAX_MSG_LOG)
    319332            {
    320                 tr_msg_list * newmsg;
    321                 newmsg = tr_new0( tr_msg_list, 1 );
    322                 newmsg->level = level;
    323                 newmsg->when = tr_time( );
    324                 newmsg->message = tr_strdup( buf );
    325                 newmsg->file = file;
    326                 newmsg->line = line;
    327                 newmsg->name = tr_strdup( name );
    328 
    329                 *messageQueueTail = newmsg;
    330                 messageQueueTail = &newmsg->next;
    331                 ++messageQueueCount;
     333                tr_msg_list * old = messageQueue;
     334                messageQueue = old->next;
     335                old->next = NULL;
     336                tr_freeMessageList(old);
    332337               
    333                 if (messageQueueCount > TR_MAX_MSG_LOG)
    334                 {
    335                     tr_msg_list * old = messageQueue;
    336                     messageQueue = old->next;
    337                     old->next = NULL;
    338                     tr_freeMessageList(old);
    339                    
    340                     --messageQueueCount;
    341                    
    342                     assert( messageQueueCount == TR_MAX_MSG_LOG );
    343                 }
     338                --messageQueueCount;
     339               
     340                assert( messageQueueCount == TR_MAX_MSG_LOG );
    344341            }
     342        }
     343        else
     344        {
     345            char timestr[64];
     346            FILE * fp;
     347
     348            fp = tr_getLog( );
     349            if( fp == NULL )
     350                fp = stderr;
     351
     352            tr_getLogTimeStr( timestr, sizeof( timestr ) );
     353
     354            if( name )
     355                fprintf( fp, "[%s] %s: %s\n", timestr, name, buf );
    345356            else
    346             {
    347                 char timestr[64];
    348                 FILE * fp;
    349 
    350                 fp = tr_getLog( );
    351                 if( fp == NULL )
    352                     fp = stderr;
    353 
    354                 tr_getLogTimeStr( timestr, sizeof( timestr ) );
    355 
    356                 if( name )
    357                     fprintf( fp, "[%s] %s: %s\n", timestr, name, buf );
    358                 else
    359                     fprintf( fp, "[%s] %s\n", timestr, buf );
    360                 fflush( fp );
    361             }
     357                fprintf( fp, "[%s] %s\n", timestr, buf );
     358            fflush( fp );
    362359        }
    363360    }
Note: See TracChangeset for help on using the changeset viewer.