已合并
fix : 【log】clean code #2111
GuoWenbo创建于 5月13日
fix : 【log】clean code #2111
已合并
GuoWenbo创建于 5月13日
1 个文件变更+19-8
@@ -16,14 +16,25 @@
16#include <stdarg.h>16#include <stdarg.h>
17#include "log_types.h"17#include "log_types.h"
18 18 
19+static inline const char *DlogGetFileName(const char *filePath) {
20+ const char *result = filePath;
21+ for (size_t i = 0; filePath[i] != '\0'; i++) {
22+ if (filePath[i] == '/') {
23+ result = &filePath[i + 1U];
24+ }
25+ }
26+ return result;
27+}
28+ 
29+#define DLOG_FILE_NAME (DlogGetFileName(__FILE__))
30+#define __FILENAME__ DLOG_FILE_NAME
31+ 
19#ifdef __cplusplus32#ifdef __cplusplus
20#ifndef LOG_CPP33#ifndef LOG_CPP
21extern "C" {34extern "C" {
22#endif35#endif
23#endif // __cplusplus36#endif // __cplusplus
24 37 
25-#define __FILENAME__ (strrchr(__FILE__, '/') ? strrchr(__FILE__, '/') + 1 : __FILE__)
26- 
27/**38/**
28 * @brief get module debug loglevel and enableEvent39 * @brief get module debug loglevel and enableEvent
29 * @param [in] moduleId moudule id(see log_types.h, eg: CCE), others: invalid40 * @param [in] moduleId moudule id(see log_types.h, eg: CCE), others: invalid
@@ -80,7 +91,7 @@ LOG_FUNC_VISIBILITY void DlogFlush(void);
80 */91 */
81#define dlog_error(moduleId, fmt, ...) \92#define dlog_error(moduleId, fmt, ...) \
82 do { \93 do { \
83- DlogRecord(moduleId, DLOG_ERROR, "[%s:%d]" fmt, __FILENAME__, __LINE__, ##__VA_ARGS__); \94+ DlogRecord(moduleId, DLOG_ERROR, "[%s:%d]" fmt, DLOG_FILE_NAME, __LINE__, ##__VA_ARGS__); \
84 } while (0)95 } while (0)
85 96 
86/**97/**
@@ -93,7 +104,7 @@ LOG_FUNC_VISIBILITY void DlogFlush(void);
93#define dlog_warn(moduleId, fmt, ...) \104#define dlog_warn(moduleId, fmt, ...) \
94 do { \105 do { \
95 if (CheckLogLevel(moduleId, DLOG_WARN) == 1) { \106 if (CheckLogLevel(moduleId, DLOG_WARN) == 1) { \
96- DlogRecord(moduleId, DLOG_WARN, "[%s:%d]" fmt, __FILENAME__, __LINE__, ##__VA_ARGS__); \107+ DlogRecord(moduleId, DLOG_WARN, "[%s:%d]" fmt, DLOG_FILE_NAME, __LINE__, ##__VA_ARGS__); \
97 } \108 } \
98 } while (0)109 } while (0)
99 110 
@@ -107,7 +118,7 @@ LOG_FUNC_VISIBILITY void DlogFlush(void);
107#define dlog_info(moduleId, fmt, ...) \118#define dlog_info(moduleId, fmt, ...) \
108 do { \119 do { \
109 if (CheckLogLevel(moduleId, DLOG_INFO) == 1) { \120 if (CheckLogLevel(moduleId, DLOG_INFO) == 1) { \
110- DlogRecord(moduleId, DLOG_INFO, "[%s:%d]" fmt, __FILENAME__, __LINE__, ##__VA_ARGS__); \121+ DlogRecord(moduleId, DLOG_INFO, "[%s:%d]" fmt, DLOG_FILE_NAME, __LINE__, ##__VA_ARGS__); \
111 } \122 } \
112 } while (0)123 } while (0)
113 124 
@@ -121,7 +132,7 @@ LOG_FUNC_VISIBILITY void DlogFlush(void);
121#define dlog_debug(moduleId, fmt, ...) \132#define dlog_debug(moduleId, fmt, ...) \
122 do { \133 do { \
123 if (CheckLogLevel(moduleId, DLOG_DEBUG) == 1) { \134 if (CheckLogLevel(moduleId, DLOG_DEBUG) == 1) { \
124- DlogRecord(moduleId, DLOG_DEBUG, "[%s:%d]" fmt, __FILENAME__, __LINE__, ##__VA_ARGS__); \135+ DlogRecord(moduleId, DLOG_DEBUG, "[%s:%d]" fmt, DLOG_FILE_NAME, __LINE__, ##__VA_ARGS__); \
125 } \136 } \
126 } while (0)137 } while (0)
127 138 
@@ -136,7 +147,7 @@ LOG_FUNC_VISIBILITY void DlogFlush(void);
136#define Dlog(moduleId, level, fmt, ...) \147#define Dlog(moduleId, level, fmt, ...) \
137 do { \148 do { \
138 if (CheckLogLevel(moduleId, level) == 1) { \149 if (CheckLogLevel(moduleId, level) == 1) { \
139- DlogRecord(moduleId, level, "[%s:%d]" fmt, __FILENAME__, __LINE__, ##__VA_ARGS__); \150+ DlogRecord(moduleId, level, "[%s:%d]" fmt, DLOG_FILE_NAME, __LINE__, ##__VA_ARGS__); \
140 } \151 } \
141 } while (0)152 } while (0)
142 153 
@@ -152,7 +163,7 @@ LOG_FUNC_VISIBILITY void DlogFlush(void);
152#define DlogSub(moduleId, submodule, level, fmt, ...) \163#define DlogSub(moduleId, submodule, level, fmt, ...) \
153 do { \164 do { \
154 if (CheckLogLevel(moduleId, level) == 1) { \165 if (CheckLogLevel(moduleId, level) == 1) { \
155- DlogRecord(moduleId, level, "[%s:%d][%s]" fmt, __FILENAME__, __LINE__, submodule, ##__VA_ARGS__); \166+ DlogRecord(moduleId, level, "[%s:%d][%s]" fmt, DLOG_FILE_NAME, __LINE__, submodule, ##__VA_ARGS__); \
156 } \167 } \
157 } while (0)168 } while (0)
158 169