Reverts Backtrace: format to what it used to be

Was:  Backtrace: N:M N:M N:M ...
Now:  Backtrace:N:MN:M N:M ...

The problem with the new format is that it is hard to parse and
breaks the parser that is used by PlatformIO.  The old format
is much more reasonable.  I do not see how the pattern in IDFDUT.py
can work with the new format, due to the missing space after the :
This commit is contained in:
Mitch Bradley
2022-06-10 18:33:46 -10:00
parent 01d014c42d
commit 8d9e110a04

View File

@@ -36,12 +36,12 @@ bool IRAM_ATTR esp_backtrace_get_next_frame(esp_backtrace_frame_t *frame)
static void IRAM_ATTR print_entry(uint32_t pc, uint32_t sp, bool panic) static void IRAM_ATTR print_entry(uint32_t pc, uint32_t sp, bool panic)
{ {
if (panic) { if (panic) {
panic_print_str("0x"); panic_print_str(" 0x");
panic_print_hex(pc); panic_print_hex(pc);
panic_print_str(":0x"); panic_print_str(":0x");
panic_print_hex(sp); panic_print_hex(sp);
} else { } else {
esp_rom_printf("0x%08X:0x%08X", pc, sp); esp_rom_printf(" 0x%08X:0x%08X", pc, sp);
} }
} }
@@ -80,7 +80,6 @@ esp_err_t IRAM_ATTR esp_backtrace_print_from_frame(int depth, const esp_backtrac
corrupted = true; corrupted = true;
} }
print_entry(esp_cpu_process_stack_pc(stk_frame.pc), stk_frame.sp, panic); print_entry(esp_cpu_process_stack_pc(stk_frame.pc), stk_frame.sp, panic);
print_str(" ", panic);
} }
//Print backtrace termination marker //Print backtrace termination marker