assert问题&elog_hex_dump
IdiotStoneMa opened this issue · comments
1、我们需要使用hex_dump功能,我参考ulog_hex_demp实现了,我可以提交一个pr吗?
2、elog_async_init中init_ok = true;建议提前到ifdef ELOG_ASYNC_OUTPUT_USING_PTHREAD宏定义前,我修改异步输出任务调度方式为SCHED_FIFO时发现,在任务还没有返回时async_output回调就被调用,会在该函数中ELOG_ASSERT(init_ok);导致无法输出。
- 欢迎 hex_dump 功能的 pr 哈
- 或者把 ELOG_ASSERT(init_ok) 删掉吧,感觉放这个 static 函数里没啥用呢
感谢回复,easylogger在嵌入式上很好用,手动点赞666.
1、 我接下来会提交pr。
2、 建议将异步输出任务创建和信号量相关也可以移动到port.c中,这部分也是需要在不同系统上进行适配的。
我们这边需要同时适配windows+linux+threadx,我本地建了3个对应的port文件,将任务和信号量部分全部移出了。
发件人: 朱天龙 (Armink) [mailto:notifications@github.com]
发送时间: 2018年12月21日 14:25
收件人: armink/EasyLogger
抄送: IdiotStoneMa; Author
主题: Re: [armink/EasyLogger] assert问题&elog_hex_dump (#31)
* 欢迎 hex_dump 功能的 pr 哈
* 或者把 ELOG_ASSERT(init_ok) 删掉吧,感觉放这个 static 函数里没啥用呢
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#31 (comment)> , or mute the thread <https://github.com/notifications/unsubscribe-auth/AfMAvlQYFQtyII92_1xIeJZwHj-M8LSvks5u7H6hgaJpZM4ZdTOO> . <https://github.com/notifications/beacon/AfMAvr9TbLcVKmYYL7u5dXBDKFzkdhOZks5u7H6hgaJpZM4ZdTOO.gif>
ELOG_ASSERT(init_ok) 的优化还做吗?
其他内容已经在 #32 中更新
1、 还在做。
2、 是否可以将异步线程创建、信号量等移动到port.c中?
发件人: 朱天龙 (Armink) [mailto:notifications@github.com]
发送时间: 2018年12月25日 21:27
收件人: armink/EasyLogger
抄送: IdiotStoneMa; Author
主题: Re: [armink/EasyLogger] assert问题&elog_hex_dump (#31)
ELOG_ASSERT(init_ok) 的优化还做吗?
其他内容已经在 #32 <#32> 中更新
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub <#31 (comment)> , or mute the thread <https://github.com/notifications/unsubscribe-auth/AfMAvhKbwldmWqGZS6wUij7wBHu7jvsZks5u8ieHgaJpZM4ZdTOO> . <https://github.com/notifications/beacon/AfMAvszYylnUIhV0l9aPd6LPBqP-syAQks5u8ieHgaJpZM4ZdTOO.gif>
也可以的,主要 elog_async.c 中提供了基于 pthread 的实现,这样用户这边可能就无需再针对异步模式再做移植了,降低了用户的一些工作量。
好的,这几天空了我就提交pr,后续有啥我这边也可以帮忙处理,这段时间会大量使用elog~
好呀,后面也可以加 QQ(3830__16632) 交流。