a1012112796

Results 20 comments of a1012112796

不知道 github action 是否像`drone ci` 一样支持使用自定义的 docker 镜像来运行ci, 如果可以,那就可以打包一个专门的镜像用于构建测试 rt-thread。 示例: https://gitea.com/gitea/test-env/src/branch/master/Dockerfile

尝试了一下基于 docker 的方案,但提速效果不明显,每次的docker 镜像似乎还是重新下载的: https://github.com/a1012112796/rt-thread/runs/7194339923?check_suite_focus=true 对比 当前: ![image](https://user-images.githubusercontent.com/25342410/177307701-4901651f-f76f-4e2f-9037-ea668d03b5a2.png) 使用 docker 后 ![image](https://user-images.githubusercontent.com/25342410/177307962-86dd4ab3-d60b-440e-a4b3-0aa82675d2f0.png)

> "The only translation that is maintained in this Git repository is [en_US.ini](https://github.com/go-gitea/gitea/blob/master/options/locale/locale_en-US.ini) and is synced regularly to Crowdin." > > Does this need another PR? I'm not sure what...

fixed by #28969 ? @yp05327 @lunny

> I've marked this as a WIP because as multiple others mentioned, pinning repositories would be better. The feature works as-is, however. If it's preferred for pinning repositories to be...

That's follow github style. see: https://github.com/go-gitea/gitea/pull/10756#discussion_r396892580

> 按照字面意思,void *user_data; 应该留给用户使用,目前在组件和软件包中都有占用; 个人认为不应该, device 的结构体中的所有成员变量都是供驱动逻辑使用的,其他组件/模块调用设备时应该只使用 rt_device_open / read/ write 等 api 完成对设备的操作,不应该直接使用设备结构体的任何成员变量。

> 不是的 这个是设计的问题,user_data这个词给用户造成了极大地误解,或者说早期阶段,rt-thread在没有过多上层建筑是 user_data确实是给user用的,但是上层建筑开始逐渐丰富起来之后,rt-thread的这些上层建筑也开始偷偷摸摸自己去用这个字段了。所以这里要重新规范一下。 - 文档注释还是比较明确的。。。 ![image](https://user-images.githubusercontent.com/25342410/210963912-f25e5a28-b6eb-45e6-a534-1779b783f55e.png) - 个人感觉可能考虑添加如下接口: ```CXX void rt_device_set_param(rt_device_t dev, void *param); void *rt_device_get_param(rt_device_t dev); ``` - 更希望可以给hook 加参数,但这样对旧的代码破坏就太大了。。。 ```DIFF diff --git a/rt-thread/include/rtthread.h b/rt-thread/include/rtthread.h index c55720b..85d9016 100644...

确实不是很明白此处设`thread->error = -RT_EINTR;` 有何作用? 或许应该把 eintr 理解为一种正常情况,就像 etimeout 一样。。。