This is an automated email from Gerrit. "Evgeniy Naydanov <evgeniy.nayda...@syntacore.com>" just uploaded a new patch set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/7223
-- gerrit commit 80562d0fec31d04294ab8795d371b54eecba6624 Author: Evgeniy Naydanov <evgeniy.nayda...@syntacore.com> Date: Wed Sep 14 17:56:20 2022 +0300 Remove duplicate of a counter in hwthread_update_threads There is no need to count number of examined threads twice. Signed-off-by: Evgeniy Naydanov <evgeniy.nayda...@syntacore.com> Change-Id: Id32ead853d1ddcd4e67062d6f795700feb20cb4b diff --git a/src/rtos/hwthread.c b/src/rtos/hwthread.c index e5eaf425f6..32b3bde328 100644 --- a/src/rtos/hwthread.c +++ b/src/rtos/hwthread.c @@ -78,7 +78,6 @@ static int hwthread_fill_thread(struct rtos *rtos, struct target *curr, int thre static int hwthread_update_threads(struct rtos *rtos) { int threads_found = 0; - int thread_list_size = 0; struct target_list *head; struct target *target; int64_t current_thread = 0; @@ -100,13 +99,13 @@ static int hwthread_update_threads(struct rtos *rtos) if (!target_was_examined(curr)) continue; - ++thread_list_size; + ++threads_found; } } else - thread_list_size = 1; + thread_found = 1; /* create space for new thread details */ - rtos->thread_details = malloc(sizeof(struct thread_detail) * thread_list_size); + rtos->thread_details = malloc(sizeof(struct thread_detail) * threads_found); if (target->smp) { /* loop over all threads */ @@ -171,13 +170,10 @@ static int hwthread_update_threads(struct rtos *rtos) default: break; } - - threads_found++; } } else { hwthread_fill_thread(rtos, target, threads_found); current_thread = threadid_from_target(target); - threads_found++; } rtos->thread_count = threads_found; --