Nithilan Rameshkumar commented: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/issues/5409#note_139893


Hi, I'm interested in working on this issue.

I've looked through the code and understand the goal: refactor 
`Stack_checker_Reporter_handler` to take a single `rtems_stack_checker_info` 
pointer instead of exposing `Thread_Control`/`rtems_tcb`.

**My approach:**

1. Extend `rtems_stack_checker_info` with a `pattern_ok` field
2. Update `Stack_checker_Reporter_handler` typedef and built-in reporters
3. Add a helper in `check.c` to populate the info struct before calling the 
reporter
4. Update the `stackchk03` test's custom reporter

**Question:** Should this be a breaking change, or is a deprecation period 
needed for the old signature?

Please let me know if this looks correct and I will proceed with the draft MR.

-- 
View it on GitLab: 
https://gitlab.rtems.org/rtems/rtos/rtems/-/issues/5409#note_139893
You're receiving this email because of your account on gitlab.rtems.org.


_______________________________________________
bugs mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/bugs

Reply via email to