Eliminate `bootstrappable.c`
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 35/100
- Issue type
- Refactor
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- c
- Domain
- operating-systems
Research direction
Read bootstrappable.c and trace the remaining require, match, in_set, and strtoint call sites. Check which standard interfaces are already available, then replace the uses and remove the obsolete functions; completion means the listed non-standard functions are gone or unused, while int2str remains as noted.
Written by the indexing model from the issue text.
Description
bootstrappable.c has a number of functions from the early years of the M2* projects that were helpful at the time. However, we have gradually moved away from these towards more standard C library interfaces (as per this project).
We should eliminate the remaining non-standard functions from bootstrappable.c, first by introducing the new interfaces where they do not already exist, then replacing existing code with the standard interfaces, then removing the old functions.
-
require->assert- implemented
- replaced
- removed
-
match->strcmp- implemented
- replaced
- removed
-
in_set->strchr- implemented
- replaced
- removed
-
strtoint->atoi- implemented
- replaced
- removed
int2stris probably not removable at this time
- Dominant language
- C
- Stars
- 27
- Forks
- 20
- Avg merge
- 5d 12h
- Merged PRs (30d)
- 2
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
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.
More from oriansj/M2libc
-
armv7l: `close`, `unlink` and `chroot` pass the kernel their argument's addressPossibly taken @aoinoikaz claimed this 5 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
`fseek` returns the new position instead of 0Possibly taken @aoinoikaz claimed this 5 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 3/5 1-2 days Newbie friendliness 62/100
-
Difficulty 5/5 Over a week Newbie friendliness 20/100
Similar issues
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
msys2/MINGW-packages#33356 · 1 comment ·
Maintainers usually reply within 1 day
-
Difficulty 1/5 1-3 hours Newbie friendliness 78/100
facebookincubator/muse-gadget-sdk#355 ·
Maintainers usually reply within 4 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
kmod-project/kmod#492 ·
Maintainers usually reply within 4 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
PecanProject/sipnet#435 · 1 comment ·
Maintainers usually reply within 2 days
-
SDL_GPU Vulkan: wrong screen orientation on OpenHarmony (preTransform set without pre-rotation)Open
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
libsdl-org/SDL#16670 ·
Maintainers usually reply within 1 day