Changeset 123 in libcf

Timestamp:

06/12/13 13:28:21 (12 years ago)

Author:

cheese

Message:

#1 fix windows bug for r122

Location:

trunk

Files:

• include/cf_error.h (modified) (1 diff)
• src/cf_log.c (modified) (22 diffs)
• src/cf_thread.c (modified) (4 diffs)

trunk/include/cf_error.h

@@ -69 +69 @@
 #define CF_ERROR_MUTEX_CREATE               CF_ERROR_MUTEX - 2
 #define CF_ERROR_MUTEX_INVALID_CTX          CF_ERROR_MUTEX - 3
+#define CF_ERROR_MUTEX_INVALID_ARGS         CF_ERROR_MUTEX - 4
 /* }}} mutex */
 

trunk/src/cf_log.c

@@ -4 +4 @@
  * \author  vfire
  *
- * \brief 로그 구현
+ * \brief ·Î±× ±¸Çö
  */
 #if defined(_WIN32) || defined(_WIN64)
@@ -59 +59 @@
 #define LOG_DATETIME_LENGTH             sizeof ("0000-00-00 00:00:00.000") - 1
 
-/** 로그 컨텍스트 (Opaque) */
+/** ·Î±× ÄÁÅؽºÆ® (Opaque) */
 typedef void *  CF_Log_Ctx;
 
@@ -75 +75 @@
 } CF_LOG_DATETIME;
 
-/** 로그 컨텍스트 (CF_Log_Ctx의 구현) */
+/** ·Î±× ÄÁÅؽºÆ® (CF_Log_CtxÀÇ ±¸Çö) */
 typedef struct __cf_log_ctx__
 {
@@ -195 +195 @@
 
 /**
- * 로그 데이터 처리
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드
- *
- * \param ctx           로그 컨텍스트
- * \param buffer        로그 데이터
- * \param demandSize    로그 데이터 길이
+ * ·Î±× µ¥ÀÌÅÍ Ã³¸®
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå
+ *
+ * \param ctx           ·Î±× ÄÁÅؽºÆ®
+ * \param buffer        ·Î±× µ¥ÀÌÅÍ
+ * \param demandSize    ·Î±× µ¥ÀÌÅÍ ±æÀÌ
  *
  * \author vfire
@@ -212 +212 @@
     int result = CF_OK;
 
-    if (ctx->size > 0) /* 버퍼단위 버퍼링.... */
+    if (ctx->size > 0) /* ¹öÆÛ´ÜÀ§ ¹öÆÛ¸µ.... */
     {
         size_t writeSize;
@@ -231 +231 @@
         }
     }
-    else /* flush되어야 함. */
+    else /* flushµÇ¾î¾ß ÇÔ. */
     {
         ctx->buffer = (char *)buffer;
@@ -243 +243 @@
 
 /**
- * 로그 컨텍스트에 멀티쓰레드 모드 설정
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드 
- *
- * \param ctx 로그 컨텍스트
+ * ·Î±× ÄÁÅؽºÆ®¿¡ ¸ÖƼ¾²·¹µå ¸ðµå ¼³Á¤
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå 
+ *
+ * \param ctx ·Î±× ÄÁÅؽºÆ®
  *
  * \see CF_Log_UnsetMultiThread
@@ -265 +265 @@
 
 /**
- * 로그 컨텍스트에 멀티쓰레드 모드 설정 해제
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드 
- *
- * \param ctx 로그 컨텍스트
+ * ·Î±× ÄÁÅؽºÆ®¿¡ ¸ÖƼ¾²·¹µå ¸ðµå ¼³Á¤ ÇØÁ¦
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå 
+ *
+ * \param ctx ·Î±× ÄÁÅؽºÆ®
  *
  * \see CF_Log_SetMultiThread
@@ -289 +289 @@
 
 /**
- * 로그 컨텍스트에 따라 로그 쓰기
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드
- *
- * \param ctx       로그 컨텍스트
- * \param prefix    로그의 프리픽스 문자열
- * \param fmt       포맷 스트링
- * \param ...       가변 인자
+ * ·Î±× ÄÁÅؽºÆ®¿¡ µû¶ó ·Î±× ¾²±â
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå
+ *
+ * \param ctx       ·Î±× ÄÁÅؽºÆ®
+ * \param prefix    ·Î±×ÀÇ ÇÁ¸®ÇȽº ¹®ÀÚ¿­
+ * \param fmt       Æ÷¸Ë ½ºÆ®¸µ
+ * \param ...       °¡º¯ ÀÎÀÚ
  */
 static int
@@ -330 +330 @@
 
 /**
- * 로그 버퍼의 데이터를 즉시 로그 파일에 쓰기
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드
- *
- * \param ctx 로그 컨텍스트
+ * ·Î±× ¹öÆÛÀÇ µ¥ÀÌÅ͸¦ Áï½Ã ·Î±× ÆÄÀÏ¿¡ ¾²±â
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå
+ *
+ * \param ctx ·Î±× ÄÁÅؽºÆ®
  */
 static int
@@ -351 +351 @@
 
 /**
- * 로그 컨텍스트 해제
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드 
- *
- * \param ctx 로그 컨텍스트
+ * ·Î±× ÄÁÅؽºÆ® ÇØÁ¦
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå 
+ *
+ * \param ctx ·Î±× ÄÁÅؽºÆ®
  */
 static int
@@ -381 +381 @@
 
 /**
- * 로그 컨텍스트 생성
- *
- * \return 성공 시, 로그 컨텍스트; 실패 시, NULL
- *
- * \param ctx       로그 컨텍스트 주소
- * \param path      로그 파일 경로
- * \param memsize   로그 버퍼 크기
+ * ·Î±× ÄÁÅؽºÆ® »ý¼º
+ *
+ * \return ¼º°ø ½Ã, ·Î±× ÄÁÅؽºÆ®; ½ÇÆÐ ½Ã, NULL
+ *
+ * \param ctx       ·Î±× ÄÁÅؽºÆ® ÁÖ¼Ò
+ * \param path      ·Î±× ÆÄÀÏ °æ·Î
+ * \param memsize   ·Î±× ¹öÆÛ Å©±â
  *
  * \see CF_LOG_DEFAULT_BUFFER, CF_LOG_NO_BUFFER
@@ -446 +446 @@
 
 /**
- * 로그 컨텍스트에 아이디 넘버 할당<br />
- * 로그 기록 시, 아이디 넘버를 사용하면 해당 로그로 기록할 수 있음
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드
- *
- * \param mapid 부여할 아이디 넘버
- * \param ctx   로그 컨텍스트
- *
- * \remarks 반드시 먼저 초기화 해야하며, 초기화 시에 주어진 번호보다 작은 아이디 넘버를 사용
+ * ·Î±× ÄÁÅؽºÆ®¿¡ ¾ÆÀ̵ð ³Ñ¹ö ÇÒ´ç<br />
+ * ·Î±× ±â·Ï ½Ã, ¾ÆÀ̵ð ³Ñ¹ö¸¦ »ç¿ëÇϸé ÇØ´ç ·Î±×·Î ±â·ÏÇÒ ¼ö ÀÖÀ½
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå
+ *
+ * \param mapid ºÎ¿©ÇÒ ¾ÆÀ̵ð ³Ñ¹ö
+ * \param ctx   ·Î±× ÄÁÅؽºÆ®
+ *
+ * \remarks ¹Ýµå½Ã ¸ÕÀú ÃʱâÈ­ ÇؾßÇϸç, ÃʱâÈ­ ½Ã¿¡ ÁÖ¾îÁø ¹øÈ£º¸´Ù ÀÛÀº ¾ÆÀ̵ð ³Ñ¹ö¸¦ »ç¿ë
  *
  * \see CF_LOG_OPEN, CF_Log_CreateCtx
@@ -472 +472 @@
 
 /**
- * 아이디 넘버에 해당하는 로그를 닫고 해당하는 컨텍스트를 해제
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드
- *
- * \param mapid 로그의 아이디 넘버
- *
- * \remarks 아이디 넘버에 해당하는 컨텍스트가 해제되므로 주의
+ * ¾ÆÀ̵ð ³Ñ¹ö¿¡ ÇØ´çÇÏ´Â ·Î±×¸¦ ´Ý°í ÇØ´çÇÏ´Â ÄÁÅؽºÆ®¸¦ ÇØÁ¦
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå
+ *
+ * \param mapid ·Î±×ÀÇ ¾ÆÀ̵ð ³Ñ¹ö
+ *
+ * \remarks ¾ÆÀ̵ð ³Ñ¹ö¿¡ ÇØ´çÇÏ´Â ÄÁÅؽºÆ®°¡ ÇØÁ¦µÇ¹Ç·Î ÁÖÀÇ
  *
  * \see CF_LOG_CLOSE, CF_Log_DestroyCtx
@@ -498 +498 @@
 
 /**
- * 아이디 넘버에 해당하는 로그 컨텍스트를 얻기
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드
- *
- * \param mapid 로그의 아이디 넘버
- * \param ctx   로그 컨텍스트 받을 주소
+ * ¾ÆÀ̵ð ³Ñ¹ö¿¡ ÇØ´çÇÏ´Â ·Î±× ÄÁÅؽºÆ®¸¦ ¾ò±â
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå
+ *
+ * \param mapid ·Î±×ÀÇ ¾ÆÀ̵ð ³Ñ¹ö
+ * \param ctx   ·Î±× ÄÁÅؽºÆ® ¹ÞÀ» ÁÖ¼Ò
  */
 static int
@@ -519 +519 @@
 
 /**
- * 로그를 사용하기 위해 초기화
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드
- *
- * \param poolSize  로그 풀 크기로, 로그 아이디 넘버의 최대 값
+ * ·Î±×¸¦ »ç¿ëÇϱâ À§ÇØ ÃʱâÈ­
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå
+ *
+ * \param poolSize  ·Î±× Ç® Å©±â·Î, ·Î±× ¾ÆÀ̵ð ³Ñ¹öÀÇ ÃÖ´ë °ª
  */
 int
@@ -543 +543 @@
 
 /**
- * 로그가 모두 사용된 후 자원 해제
- *
- * \return CF_OK 반환
+ * ·Î±×°¡ ¸ðµÎ »ç¿ëµÈ ÈÄ ÀÚ¿ø ÇØÁ¦
+ *
+ * \return CF_OK ¹Ýȯ
  */
 int
@@ -566 +566 @@
 
 /**
- * 로그 열기
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드
- *
- * \param mapid     로그의 아이디 넘버
- * \param path      로그 파일 경로
- * \param memsize   로그 버퍼 크기
+ * ·Î±× ¿­±â
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå
+ *
+ * \param mapid     ·Î±×ÀÇ ¾ÆÀ̵ð ³Ñ¹ö
+ * \param path      ·Î±× ÆÄÀÏ °æ·Î
+ * \param memsize   ·Î±× ¹öÆÛ Å©±â
  *
  * \see CF_LOG_DEFAULT_BUFFER, CF_LOG_NO_BUFFER
@@ -596 +596 @@
 
 /**
- * 로그 닫기
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드
- *
- * \param mapid 로그의 아이디 넘버
+ * ·Î±× ´Ý±â
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå
+ *
+ * \param mapid ·Î±×ÀÇ ¾ÆÀ̵ð ³Ñ¹ö
  */
 int
@@ -609 +609 @@
 
 /**
- * 로그 버퍼의 데이터를 즉시 로그 파일에 쓰기
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드
- *
- * \param mapid 로그의 아이디 넘버
+ * ·Î±× ¹öÆÛÀÇ µ¥ÀÌÅ͸¦ Áï½Ã ·Î±× ÆÄÀÏ¿¡ ¾²±â
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå
+ *
+ * \param mapid ·Î±×ÀÇ ¾ÆÀ̵ð ³Ñ¹ö
  */
 int
@@ -631 +631 @@
 
 /**
- * 로그 컨텍스트에 멀티쓰레드 모드 설정
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드 
- *
- * \param mapid 로그의 아이디 넘버
- * \param flag  설정/해제 bool 플래그
+ * ·Î±× ÄÁÅؽºÆ®¿¡ ¸ÖƼ¾²·¹µå ¸ðµå ¼³Á¤
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå 
+ *
+ * \param mapid ·Î±×ÀÇ ¾ÆÀ̵ð ³Ñ¹ö
+ * \param flag  ¼³Á¤/ÇØÁ¦ bool Ç÷¡±×
  *
  * \see CF_BOOL
@@ -658 +658 @@
 
 /**
- * 로그 쓰기
- *
- * \return 성공 시, CF_OK; 실패 시, 오류 코드
- *
- * \param mapid     로그의 아이디 넘버
- * \param prefix    로그의 프리픽스 문자열
- * \param fmt       포맷 스트링
- * \param ...       가변 인자
+ * ·Î±× ¾²±â
+ *
+ * \return ¼º°ø ½Ã, CF_OK; ½ÇÆÐ ½Ã, ¿À·ù ÄÚµå
+ *
+ * \param mapid     ·Î±×ÀÇ ¾ÆÀ̵ð ³Ñ¹ö
+ * \param prefix    ·Î±×ÀÇ ÇÁ¸®ÇȽº ¹®ÀÚ¿­
+ * \param fmt       Æ÷¸Ë ½ºÆ®¸µ
+ * \param ...       °¡º¯ ÀÎÀÚ
  */
 int

trunk/src/cf_thread.c

@@ -52 +52 @@
 {
 #if defined(_WIN32) || defined(_WIN64)
+    if (tid == NULL)
+        return CF_ERROR_THREAD_INVALID_ARGS;
+
     CloseHandle (tid);
 #endif
@@ -62 +65 @@
 {
 #if defined(_WIN32) || defined(_WIN64)
+    if (mid == NULL)
+        return CF_ERROR_MUTEX_INVALID_ARGS;
+
     CloseHandle (mid);
 #else
@@ -230 +236 @@
 #if defined(_WIN32) || defined(_WIN64)
         context->mid = CreateMutexA (NULL, FALSE, NULL);
-        if (*ctx == NULL)
+        if (context->mid == NULL)
         {
             result = CF_ERROR_MUTEX_CREATE;
@@ -243 +249 @@
         }
 #endif
+
+        *ctx = context;
     }
     CATCH_IF (result < 0)
     {
-        CF_Mutex_DestoryCtx (ctx);
+        CF_Mutex_DestoryCtx ((CF_Mutex_Ctx) context);
     }
 
-    return CF_OK;
+    return result;
 }