diff options
| author | Frédéric Desbiens <[email protected]> | 2026-08-09 10:12:09 -0400 |
|---|---|---|
| committer | GitHub <[email protected]> | 2026-08-09 10:12:09 -0400 |
| commit | cbdf924d6bca6463ee78ba0f6ebfc47697ea018a (patch) | |
| tree | 561be846175500329c3ccaab83bd8df78a7d6279 /common_modules/module_lib/src/txm_thread_delete.c | |
| parent | cfed1d8095f7f0c32f74570c5df7a47655241bb4 (diff) | |
Removed the duplicated function body in the Cortex-M4 AC6 port (#589)
_tx_thread_system_return_inline() in the Cortex-M4 AC6 tx_port.h was followed
by a second, orphaned copy of its own body. The copy had no function header, so
it declared interrupt_save at file scope and then placed statements there,
which does not compile. It is also the older version of the body, without the
dsb and isb barriers, so it was left behind rather than intended: the barriers
were added by commit 33efad3f and the previous text was not removed.
Delete the orphaned copy. What remains is the same body every sibling port
carries: after this change ports/cortex_m4/ac6/inc/tx_port.h differs from
ports/cortex_m4/iar/inc/tx_port.h and ports/cortex_m7/ac6/inc/tx_port.h only in
the port name in the banner and the version string, as it should.
Verified by compiling the function in isolation, which fails on the file scope
statements before the change and is clean afterwards, and by a structural scan
of all 208 tx_port.h files in the repository confirming this was the only
occurrence.
Fixes https://github.com/eclipse-threadx/threadx/issues/569
Assisted-by: Claude Code (Opus 5) <[email protected]>
Diffstat (limited to 'common_modules/module_lib/src/txm_thread_delete.c')
0 files changed, 0 insertions, 0 deletions