RT-Thread / RT-Thread/rt-thread
[libc] dirent.h 包含非 POSIX 定义内容
Open
Nobody has claimed this yet.
- Dominant language
- C
- Stars
- 12.2k
- Forks
- 5.4k
- Avg merge
- 4d 12h
- Merged PRs (30d)
- 40
Description
这部分宏定义并不是 POSIX 的内容,而是 DFS 内部使用的定义。
其主要使用在 dfs 内部。
且第一次出现应该也是 dfsv1/dfs.h 中。
基于以上原因,是否考虑移除并整理到 dfsv2/dfs.h 呢?
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by inspecting components/libc/compilers/common/include/dirent.h lines 19-31 and trace the referenced uses in components/dfs/dfs_v2/filesystems/cromfs/dfs_cromfs.c and components/dfs/dfs_v1/include/dfs.h. Confirm which definitions are internal rather than POSIX, then check all DFS references before deciding how to reorganize them. Done means the public dirent.h contains only appropriate definitions and DFS builds and tests remain valid.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- c
- Domain
- embedded-iot, operating-systems
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100