This is an automated email from Gerrit.

Tomas Vanek ([email protected]) just uploaded a new patch set to Gerrit, which you 
can find at http://openocd.zylin.com/5326

-- gerrit

commit e64c1e1e459639fcb5b8a01596f3e391114ed824
Author: Tomas Vanek <[email protected]>
Date:   Sun Oct 20 10:08:07 2019 +0200

    target/armv7m: cosmetic refactorization
    
    Introduce a variable 'size' and reduce the number of dereferencing
    *reg_list_size by using the variable.
    
    Change-Id: I3bdf1485a4ed8e34435e8acb3efd0df8d802508c
    Signed-off-by: Tomas Vanek <[email protected]>

diff --git a/src/target/armv7m.c b/src/target/armv7m.c
index 4b37774..2f7f4ab 100644
--- a/src/target/armv7m.c
+++ b/src/target/armv7m.c
@@ -301,20 +301,22 @@ int armv7m_get_gdb_reg_list(struct target *target, struct 
reg **reg_list[],
                int *reg_list_size, enum target_register_class reg_class)
 {
        struct armv7m_common *armv7m = target_to_armv7m(target);
-       int i;
+       int i, size;
 
        if (reg_class == REG_CLASS_ALL)
-               *reg_list_size = armv7m->arm.core_cache->num_regs;
+               size = armv7m->arm.core_cache->num_regs;
        else
-               *reg_list_size = ARMV7M_NUM_CORE_REGS;
+               size = ARMV7M_NUM_CORE_REGS;
 
-       *reg_list = malloc(sizeof(struct reg *) * (*reg_list_size));
+       *reg_list = malloc(sizeof(struct reg *) * size);
        if (*reg_list == NULL)
                return ERROR_FAIL;
 
-       for (i = 0; i < *reg_list_size; i++)
+       for (i = 0; i < size; i++)
                (*reg_list)[i] = &armv7m->arm.core_cache->reg_list[i];
 
+       *reg_list_size = size;
+
        return ERROR_OK;
 }
 

-- 


_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to