Fix code review.

Author: shibd

examples/SampleCustomLoggerCApi.c

@@ -19,29 +19,26 @@
 
 #include <pulsar/c/client.h>
 #include <stdio.h>
+#include <stdlib.h>
 #include <string.h>
 #include <time.h>
 
-void format_time(char *output){
-    time_t rawtime;
-    struct tm * timeinfo;
-
-    time(&rawtime);
-    timeinfo = localtime(&rawtime);
-
-    sprintf(output, "%d %d %d %d:%d:%d",
-            timeinfo->tm_year + 1900, timeinfo->tm_mon + 1,  timeinfo->tm_mday,
-            timeinfo->tm_hour, timeinfo->tm_min, timeinfo->tm_sec);
+char* current_time(){
+    char *time_str = malloc(128);
+    struct tm *p;
+    time_t now = time(0);
+    p=gmtime(&now);
+    strftime(time_str, 128, "%Y-%m-%d %H:%M:%S", p);
+    return time_str;
 }
 
 void custom_logger(pulsar_logger_level_t level, const char *file, int line, const char *message,
               void *ctx) {
-    time_t mytime = time(NULL);
-    char * time_str = ctime(&mytime);
     // Control the log level yourself.
     if (level >= pulsar_DEBUG) {
-        format_time(time_str);
+        char * time_str = current_time();
         printf("[%s] [%u] [%s] [%d] [%s] \n", time_str, level, file, line, message);
+        free(time_str);
     }
 }
 

lib/c/c_ClientConfiguration.cc

@@ -78,7 +78,7 @@ class PulsarCLogger : public pulsar::Logger {
     PulsarCLogger(const std::string &file, pulsar_logger logger, void *ctx)
         : file_(file), logger_(logger), ctx_(ctx) {}
 
-    bool isEnabled(Level level) { return level >= pulsar::Logger::LEVEL_DEBUG; }
+    bool isEnabled(Level level) { return true; }
 
     void log(Level level, int line, const std::string &message) {
         logger_((pulsar_logger_level_t)level, file_.c_str(), line, message.c_str(), ctx_);