This is an automated email from Gerrit.

"Antonio Borneo <borneo.anto...@gmail.com>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/6837

-- gerrit

commit ed6b861b254a1911876ea13a6c7adc4973dca4f4
Author: Antonio Borneo <borneo.anto...@gmail.com>
Date:   Mon Jan 31 11:34:12 2022 +0100

    log: drop global current_time
    
    The value of this variable is not shared across functions, so the
    variable can be local.
    
    Change-Id: I00b0444209e81c07bb57fb732f47052ad0596728
    Signed-off-by: Antonio Borneo <borneo.anto...@gmail.com>

diff --git a/src/helper/log.c b/src/helper/log.c
index 686560742..12f1790e9 100644
--- a/src/helper/log.c
+++ b/src/helper/log.c
@@ -47,7 +47,6 @@ static FILE *log_output;
 static struct log_callback *log_callbacks;
 
 static int64_t last_time;
-static int64_t current_time;
 
 static int64_t start;
 
@@ -440,8 +439,7 @@ static void gdb_timeout_warning(int64_t delta_time)
 
 void keep_alive(void)
 {
-       current_time = timeval_ms();
-
+       int64_t current_time = timeval_ms();
        int64_t delta_time = current_time - last_time;
 
        if (delta_time > KEEP_ALIVE_TIMEOUT_MS) {
@@ -469,7 +467,7 @@ void keep_alive(void)
 /* reset keep alive timer without sending message */
 void kept_alive(void)
 {
-       current_time = timeval_ms();
+       int64_t current_time = timeval_ms();
 
        int64_t delta_time = current_time - last_time;
 

-- 

Reply via email to