assert问题&elog_hex_dump

[IdiotStoneMa]

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);导致无法输出。

[armink]

• 欢迎 hex_dump 功能的 pr 哈
• 或者把 ELOG_ASSERT(init_ok) 删掉吧，感觉放这个 static 函数里没啥用呢

[IdiotStoneMa]

感谢回复，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>

[armink]

ELOG_ASSERT(init_ok) 的优化还做吗？

其他内容已经在 #32 中更新

[IdiotStoneMa]

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>

[armink]

也可以的，主要 elog_async.c 中提供了基于 pthread 的实现，这样用户这边可能就无需再针对异步模式再做移植了，降低了用户的一些工作量。

[IdiotStoneMa]

好的，这几天空了我就提交pr，后续有啥我这边也可以帮忙处理，这段时间会大量使用elog~

[armink]

好呀，后面也可以加 QQ(3830__16632) 交流。