Signed-off-by: Felipe Astroza <[email protected]>
---
 src/include/mk_clock.h |    2 ++
 src/mk_clock.c         |   13 ++++++-------
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/src/include/mk_clock.h b/src/include/mk_clock.h
index 5f57fa7..7edf3d0 100644
--- a/src/include/mk_clock.h
+++ b/src/include/mk_clock.h
@@ -34,6 +34,8 @@ mk_pointer log_current_time;
 mk_pointer header_current_time;
 
 #define GMT_DATEFORMAT "%a, %d %b %Y %H:%M:%S GMT\r\n"
+#define HEADER_TIME_BUFFER_SIZE 32
+#define LOG_TIME_BUFFER_SIZE 30
 
 void *mk_clock_worker_init(void *args);
 void mk_clock_set_time(void);
diff --git a/src/mk_clock.c b/src/mk_clock.c
index 4a75fbd..0c75f10 100644
--- a/src/mk_clock.c
+++ b/src/mk_clock.c
@@ -36,8 +36,8 @@ static void mk_clock_log_set_time()
     time_t utime;
 
     if (!log_current_time.data) {
-        log_current_time.data = mk_mem_malloc_z(30);
-        log_current_time.len = 28;
+        log_current_time.data = mk_mem_malloc_z(LOG_TIME_BUFFER_SIZE);
+        log_current_time.len = LOG_TIME_BUFFER_SIZE - 2;
     }
 
     if ((utime = time(NULL)) == -1) {
@@ -45,24 +45,23 @@ static void mk_clock_log_set_time()
     }
 
     log_current_utime = utime;
-    strftime(log_current_time.data, 30, "[%d/%b/%G %T %z]",
+    strftime(log_current_time.data, LOG_TIME_BUFFER_SIZE, "[%d/%b/%G %T %z]",
              (struct tm *) localtime((time_t *) & utime));
 }
 
 void mk_clock_header_set_time()
 {
-    int len = 32;
     time_t date;
     struct tm *gmt_tm;
 
     if (!header_current_time.data) {
-        header_current_time.data = mk_mem_malloc_z(len);
-        header_current_time.len = len - 1;
+        header_current_time.data = mk_mem_malloc_z(HEADER_TIME_BUFFER_SIZE);
+        header_current_time.len = HEADER_TIME_BUFFER_SIZE - 1;
     }
 
     date = time(NULL);
     gmt_tm = (struct tm *) gmtime(&date);
-    strftime(header_current_time.data, len, GMT_DATEFORMAT, gmt_tm);
+    strftime(header_current_time.data, HEADER_TIME_BUFFER_SIZE, GMT_DATEFORMAT, gmt_tm);
 }
 
 void *mk_clock_worker_init(void *args)
_______________________________________________
Monkey mailing list
[email protected]
http://lists.monkey-project.com/listinfo/monkey

Reply via email to