michaelh/zephyr
1
0
Fork 0
Code Releases Activity

arch: arm: core: aarch32: cortex_m: debug: remove if...else if construct

Browse source 
z_arm_debug_monitor_event_error_check() was missing final
else statement in the if else if construct so violated guideline
15.7. This commit removes the else if for symmetry in the limited
early-exit conditions, rather than empty final else {}, to comply.

Signed-off-by: Jennifer Williams <jennifer.m.williams@intel.com>
This commit is contained in:
Jennifer Williams 2021-03-10 03:37:49 +02:00 • committed by Anas Nashif
commit a5c27d69b5
1 changed files with 2 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

3
arch/arm/core/aarch32/cortex_m/debug.c
View file

@ -37,7 +37,8 @@ bool z_arm_debug_monitor_event_error_check(void)
"MATCHED flag should have been cleared on read."); "MATCHED flag should have been cleared on read.");
return true; return true;
} else if (SCB->DFSR & SCB_DFSR_BKPT_Msk) { }
if (SCB->DFSR & SCB_DFSR_BKPT_Msk) {
/* Treat BKPT events as an error as well (since they /* Treat BKPT events as an error as well (since they
* would mean the system would be stuck in an infinite loop). * would mean the system would be stuck in an infinite loop).
*/ */