hmi-debug.h:Optimise code quality 86/22186/6
authorLi Xiaoming <lixm.fnst@cn.fujitsu.com>
Wed, 21 Aug 2019 05:59:29 +0000 (13:59 +0800)
committerLi Xiaoming <lixm.fnst@cn.fujitsu.com>
Wed, 21 Aug 2019 05:59:29 +0000 (13:59 +0800)
This patch do below things:
1) standarize code indent and style;
2) add log when message is NULL, it may be a result of vasprintf failed or disable message manually;
3) add microsecond round up check.

Bug-AGL: SPEC-2760

Change-Id: I1cac7a8f4755526c40f83aa3d6dfb3fbe0819fac
Signed-off-by: Li Xiaoming <lixm.fnst@cn.fujitsu.com>
homescreen/src/hmi-debug.h

index 28705f5..47ae1f7 100644 (file)
@@ -45,26 +45,35 @@ static char ERROR_FLAG[6][20] = {"NONE", "ERROR", "WARNING", "NOTICE", "INFO", "
 
 static void _HMI_LOG(enum LOG_LEVEL level, const char* file, const char* func, const int line, const char* prefix, const char* log, ...)
 {
-    const int log_level = (getenv("USE_HMI_DEBUG") == NULL)?LOG_LEVEL_ERROR:atoi(getenv("USE_HMI_DEBUG"));
-    if(log_level < level)
-    {
+    char *message;
+    struct timespec tp;
+    uint32_t time;
+    va_list args;
+    int ret;
+    const int log_level = (getenv("USE_HMI_DEBUG") == NULL) ? LOG_LEVEL_ERROR : atoi(getenv("USE_HMI_DEBUG"));
+
+    if(log_level < level) {
         return;
     }
 
-    char *message;
-    struct timespec tp;
-    unsigned int time;
+    va_start(args, log);
+    if (vasprintf(&message, log, args) < 0) {
+       fprintf(stderr, "Warning: message is NULL\n");
+       vfprintf(stderr, log, args);
+       fprintf(stderr, "\n");
+       message = NULL;
+    }
 
     clock_gettime(CLOCK_REALTIME, &tp);
-       time = (tp.tv_sec * 1000000L) + (tp.tv_nsec / 1000);
+    time = (tp.tv_sec * 1000000L) + (tp.tv_nsec / 1000);
+    if (tp.tv_nsec % 1000 >= 500) {
+        time++;
+    }
 
-       va_list args;
-       va_start(args, log);
-       if (log == NULL || vasprintf(&message, log, args) < 0)
-        message = NULL;
     fprintf(stderr,  "[%10.3f] [%s %s] [%s, %s(), Line:%d] >>> %s \n", time / 1000.0, prefix, ERROR_FLAG[level], file, func, line, message);
+
     va_end(args);
-       free(message);
+    free(message);
 }
 
 #endif  //__HMI_DEBUG_H__