bug amend

[caoliuchao]

可能有些地方有一点问题，在UCOSII中好像找不到宏定义OS_TASK_NAME_SIZE，

在static void get_cur_thread_stack_info(uint32_t sp, uint32_t start_addr, size_t size）函数头中， 似乎 uint32_t sp 并没有什么作用。

/ the stack has S0~S15 and FPSCR registers when statck_has_fpu_regs is true, double word align / return statck_has_fpu_regs == true ? sp + sizeof(size_t) * 18 : sp; 不知道这个是18还是应该是17，可能压入了其它寄存器。

[armink]

感谢贡献

• 1、OS_TASK_NAME_SIZE 定义可以先参考 demo 文件下的 ucos 对应 demo
• 2、get_cur_thread_stack_info 中的 SP 确实没用上，但去掉之后，得把所有调用者的代码都得整理下
• 3、S0~S15 and FPSCR 这么数下来确实是 17，你有测试过改前和改后效果吗？

[armink]

1、OS_TASK_NAME_SIZE 定义可以先参考 demo 文件下的 ucos 对应 demo

这一点先按照我的建议去试下

我在百度上搜索确实是18个寄存器压入堆栈，不过你就列出17个，建议你注释完整。我感觉你可合并一些东西，感觉你的源代码不利于使用者学习。

开源的软件本身就是大家一起来玩的，如果觉得哪里不合适。可以给出依据，而不是猜测，然后提 pr 就好了。

我根据自己的习惯大改了你的代码，不知道我非商业行为发布，会不会侵权。我是一个学生。

项目本身许可非常宽泛，欢迎开发者来修改，没有侵权问题

[caoliuchao]

好的，我就根据自己的习惯修改一些代码规范，不过我不会提交合并的。 对于需要修改的代码，我会考察确定之后，提交合并需要修改的代码。 好的，很高兴看到这个软件代码。给我带来很好的效果。谢谢你了。

------------------ 原始邮件 ------------------ 发件人: "fallstool"<notifications@github.com>; 发送时间: 2020年3月20日(星期五) 下午3:34 收件人: "armink/CmBacktrace"<CmBacktrace@noreply.github.com>; 抄送: "梦中尘"<2930650069@qq.com>;"Author"<author@noreply.github.com>; 主题: Re: [armink/CmBacktrace] bug amend (#28)

1、OS_TASK_NAME_SIZE 定义可以先参考 demo 文件下的 ucos 对应 demo

这一点先按照我的建议去试下

我在百度上搜索确实是18个寄存器压入堆栈，不过你就列出17个，建议你注释完整。我感觉你可合并一些东西，感觉你的源代码不利于使用者学习。

开源的软件本身就是大家一起来玩的，如果觉得哪里不合适。可以给出依据，而不是猜测，然后提 pr 就好了。

我根据自己的习惯大改了你的代码，不知道我非商业行为发布，会不会侵权。我是一个学生。

项目本身许可非常宽泛，欢迎开发者来修改，没有侵权问题

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or unsubscribe.

[armink]

期待后面有新的进展