EasyLogger
EasyLogger copied to clipboard
armink
assert问题&elog_hex_dump
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);导致无法输出。
感谢回复，easylogger在嵌入式上很好用，手动点赞666.
1、 我接下来会提交pr。
2、 建议将异步输出任务创建和信号量相关也可以移动到port.c中，这部分也是需要在不同系统上进行适配的。
我们这边需要同时适配windows+linux+threadx，我本地建了3个对应的port文件，将任务和信号量部分全部移出了。
发件人: 朱天龙 (Armink) [mailto:[email protected]] 发送时间: 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 https://github.com/armink/EasyLogger/issues/31#issuecomment-449272632 , or mute the thread https://github.com/notifications/unsubscribe-auth/AfMAvlQYFQtyII92_1xIeJZwHj-M8LSvks5u7H6hgaJpZM4ZdTOO . https://github.com/notifications/beacon/AfMAvr9TbLcVKmYYL7u5dXBDKFzkdhOZks5u7H6hgaJpZM4ZdTOO.gif
1、 还在做。
2、 是否可以将异步线程创建、信号量等移动到port.c中？
发件人: 朱天龙 (Armink) [mailto:[email protected]] 发送时间: 2018年12月25日 21:27 收件人: armink/EasyLogger 抄送: IdiotStoneMa; Author 主题: Re: [armink/EasyLogger] assert问题&elog_hex_dump (#31)
ELOG_ASSERT(init_ok) 的优化还做吗？
其他内容已经在 #32 https://github.com/armink/EasyLogger/pull/32 中更新
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/armink/EasyLogger/issues/31#issuecomment-449850415 , or mute the thread https://github.com/notifications/unsubscribe-auth/AfMAvhKbwldmWqGZS6wUij7wBHu7jvsZks5u8ieHgaJpZM4ZdTOO . https://github.com/notifications/beacon/AfMAvszYylnUIhV0l9aPd6LPBqP-syAQks5u8ieHgaJpZM4ZdTOO.gif
也可以的，主要 elog_async.c 中提供了基于 pthread 的实现，这样用户这边可能就无需再针对异步模式再做移植了，降低了用户的一些工作量。