Changeset 85 in libcf


Ignore:
Timestamp:
05/03/13 11:16:57 (11 years ago)
Author:
cheese
Message:

#1 modify assertion logic

Location:
trunk/src
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/cf_codec.c

    r73 r85  
    99#include <stdio.h>
    1010
    11 #define ASSERT_ARGS(x)                  \
    12     if ((x))                            \
     11#define ASSERT_ARGS(x)  \
     12    if ((x))            \
    1313        return CF_ERROR_CODEC_INVALID_ARGS
    1414

  • trunk/src/cf_debug.c

    r80 r85  
    2121#define IS_READABLE_CHAR(__c)       (' ' <= __c && __c <= '~')
    2222
    23 #define ASSERT_CTX(__ctx)                   \
    24     if (__ctx == NULL)                      \
     23#define ASSERT_CTX(__ctx)   \
     24    if (__ctx == NULL)      \
    2525        return CF_ERROR_DEBUG_INVALID_CTX
    2626

  • trunk/src/cf_file.c

    r66 r85  
    4747#define FILE_MODE       S_IWUSR|S_IRUSR|S_IRGRP|S_IROTH
    4848
    49 #define ASSERT_ARGS(x)                  \
    50     if ((x))                            \
     49#define ASSERT_ARGS(x)  \
     50    if ((x))            \
    5151        return CF_ERROR_FILE_INVALID_ARGS
    5252

  • trunk/src/cf_log.c

    r67 r85  
    2525#endif
    2626
    27 #define ASSERT_CTX(__ctx)               (__ctx == NULL)
     27#define ASSERT_CTX(__ctx)   \
     28    if (__ctx == NULL)      \
     29        return CF_ERROR_LOG_INVALID_CTX
     30
     31#define ASSERT_INIT()                   \
     32    if (gLogArray.ctxPool == NULL   ||  \
     33        gLogArray.ctxSize <= 0      )   \
     34        return CF_ERROR_LOG_NOT_INITIALIZE
     35
     36#define ASSERT_MAPID(__mapid)           \
     37    if (gLogArray.ctxSize <= __mapid)   \
     38        return CF_ERROR_LOG_INVALID_MAPID
     39
     40#define ASSERT_MAPPED_CTX(__mapid)          \
     41    if (gLogArray.ctxPool[__mapid] != NULL) \
     42        return CF_ERROR_LOG_ALREADY_MAPPED_ID
     43
     44#define ASSERT_NOT_MAPPED_CTX(__mapid)      \
     45    if (gLogArray.ctxPool[__mapid] == NULL) \
     46        return CF_ERROR_LOG_NOT_MAPPED_ID
     47
    2848#define LOCK_LOG_CTX(__ctx)             CF_Mutex_Lock (&__ctx->mutex)
    2949#define UNLOCK_LOG_CTX(__ctx)           CF_Mutex_Unlock (&__ctx->mutex)
    30 #define CHECK_INITIALIZED()             (gLogArray.ctxPool == NULL||    \
    31                                          gLogArray.ctxSize <= 0 )
    32 #define ASSERT_MAPID(__mapid)           (gLogArray.ctxSize <= __mapid)
    33 #define ASSERT_MAPPED_CTX(__mapid)      (gLogArray.ctxPool[__mapid] != NULL)
    3450
    3551#define LOG_BUFFER_DEFAULT_SIZE         128 * 1024
    … …  
    235251    CF_LOG_CTX * context = (CF_LOG_CTX *) ctx;
    236252
    237     if (ASSERT_CTX (ctx))
    238         return CF_ERROR_LOG_INVALID_CTX;
     253    ASSERT_CTX (ctx);
    239254
    240255    if (CF_Mutex_Create (&context->mutex) < 0)
    … …  
    258273    CF_LOG_CTX * context = (CF_LOG_CTX *) ctx;
    259274
    260     if (ASSERT_CTX (ctx))
    261         return CF_ERROR_LOG_INVALID_CTX;
     275    ASSERT_CTX (ctx);
    262276
    263277    if (CF_Mutex_Destory (&context->mutex) < 0)
    … …  
    289303    char        datetime[LOG_DATETIME_LENGTH + 1] = {0x00,};
    290304
    291     if (ASSERT_CTX (ctx))
    292         return CF_ERROR_LOG_INVALID_CTX;
     305    ASSERT_CTX (ctx);
    293306
    294307    LOCK_LOG_CTX (context);
    … …  
    319332    CF_LOG_CTX * context = (CF_LOG_CTX *) ctx;
    320333
    321     if (ASSERT_CTX (ctx))
    322         return CF_ERROR_LOG_INVALID_CTX;
     334    ASSERT_CTX (ctx);
    323335
    324336    LOCK_LOG_CTX (context);
    … …  
    341353    CF_LOG_CTX * context = (CF_LOG_CTX *) ctx;
    342354
    343     if (ASSERT_CTX (ctx))
    344         return CF_ERROR_LOG_INVALID_CTX;
     355    ASSERT_CTX (ctx);
    345356
    346357    if (context->size > 0)
    … …  
    441452                 const CF_Log_Ctx   ctx)
    442453{
    443     int result = 0;
    444 
    445     TRY
    446     {
    447         if (CHECK_INITIALIZED ())
    448         {
    449             result = CF_ERROR_LOG_NOT_INITIALIZE;
    450             TRY_BREAK;
    451         }
    452         if (ASSERT_MAPID (mapid))
    453         {
    454             result = CF_ERROR_LOG_INVALID_MAPID;
    455             TRY_BREAK;
    456         }
    457         if (ASSERT_MAPPED_CTX (mapid))
    458         {
    459             result = CF_ERROR_LOG_ALREADY_MAPPED_ID;
    460             TRY_BREAK;
    461         }
    462     }
    463     CATCH_IF (result < 0)
    464     {
    465         CF_Log_DestroyCtx (ctx);
    466         return result;
    467     }
     454    ASSERT_INIT ();
     455    ASSERT_MAPID (mapid);
     456    ASSERT_MAPPED_CTX (mapid);
    468457
    469458    gLogArray.ctxPool[mapid] = ctx;
    … …  
    486475CF_Log_UnmapCtxID (const int mapid)
    487476{
    488     if (CHECK_INITIALIZED ())
    489         return CF_ERROR_LOG_NOT_INITIALIZE;
    490     if (ASSERT_MAPID (mapid))
    491         return CF_ERROR_LOG_INVALID_MAPID;
    492     if (!ASSERT_MAPPED_CTX (mapid))
    493         return CF_ERROR_LOG_NOT_MAPPED_ID;
     477    ASSERT_INIT ();
     478    ASSERT_MAPID (mapid);
     479    ASSERT_NOT_MAPPED_CTX (mapid);
    494480
    495481    CF_Log_DestroyCtx (gLogArray.ctxPool[mapid]);
    … …  
    513499                     CF_Log_Ctx * ctx)
    514500{
    515     if (CHECK_INITIALIZED ())
    516         return CF_ERROR_LOG_NOT_INITIALIZE;
    517     if (ASSERT_MAPID (mapid))
    518         return CF_ERROR_LOG_INVALID_MAPID;
    519     if (!ASSERT_MAPPED_CTX (mapid))
    520         return CF_ERROR_LOG_NOT_MAPPED_ID;
     501    ASSERT_INIT ();
     502    ASSERT_MAPID (mapid);
     503    ASSERT_NOT_MAPPED_CTX (mapid);
    521504
    522505    *ctx = gLogArray.ctxPool[mapid];
    … …  
    596579
    597580    result = CF_Log_MapCtxID (mapid, ctx);
     581    if (result < 0)
     582        CF_Log_DestroyCtx (ctx);
    598583
    599584    return result;

  • trunk/src/cf_socket.c

    r75 r85  
    3030#endif
    3131
    32 #define ASSERT_INIT()                       \
    33     if (!CF_Socket_IsInitialized ())        \
     32#define ASSERT_INIT()                   \
     33    if (!CF_Socket_IsInitialized ())    \
    3434        return CF_ERROR_SOCKET_NOT_INITIALIZED
    3535
    36 #define ASSERT_SOCKET(__sock)           \
    37     if (__sock < 0)                     \
     36#define ASSERT_SOCKET(__sock)   \
     37    if (__sock < 0)             \
    3838        return CF_ERROR_SOCKET_INVALID_SOCKET
    3939

  • trunk/src/cf_thread.c

    r84 r85  
    2424#endif // #if defined(_WIN32) || defined(_WIN64)
    2525
    26 #define ASSERT_THREAD(__h)              \
    27     if (__h == NULL)                    \
     26#define ASSERT_THREAD(__h)  \
     27    if (__h == NULL)        \
    2828        return CF_ERROR_THREAD_INVALID_ARGS
    2929
    30 #define ASSERT_MUTEX(__h)               \
    31     if (__h == NULL)                    \
     30#define ASSERT_MUTEX(__h)   \
     31    if (__h == NULL)        \
    3232        return CF_ERROR_MUTEX_INVALID_ARGS
    3333
Note: See TracChangeset for help on using the changeset viewer.