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) 交流。