Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

libc: implement setvbuf() #66951

Open
ycsin opened this issue Dec 24, 2023 · 2 comments
Open

libc: implement setvbuf() #66951

ycsin opened this issue Dec 24, 2023 · 2 comments
Assignees
@stephanosio
Labels
area: C Library C Standard Library Enhancement Changes/Updates/Additions to existing features Good first issue Good for a first time contributor to take

Comments

@ycsin
Copy link
Member

ycsin commented Dec 24, 2023 •
edited by cfriedt
Loading

This was originally added as a ticket under POSIX, but setvbuf() is not a POSIX function. It is from C89.

https://en.cppreference.com/w/c/io/setvbuf

The expectation from the POSIX API is that this function is implemented as part of whatever C library is in use.
@ycsin ycsin added Feature Request A request for a new feature Good first issue Good for a first time contributor to take area: POSIX POSIX API Library labels Dec 24, 2023
@ycsin ycsin mentioned this issue Dec 24, 2023
Open
@nashif nashif added Enhancement Changes/Updates/Additions to existing features and removed Feature Request A request for a new feature labels Jan 19, 2024
@whythefyouflying
Copy link

@cfriedt I would like to work on this issue if possible, I am new to Zephyr and would love some tips on how to get this issue done.

@cfriedt
Copy link
Member

cfriedt commented Feb 11, 2024

@whythefyouflying, I would suggest designing the testsuite, based on the specification. Sort of like test driven development.

Judging by the spec, setvbuf() might depend on fopen(),fclose(), and possibly others. So perhaps it's best to start on a different feature.

@cfriedt cfriedt added area: C Library C Standard Library and removed area: POSIX POSIX API Library labels Jun 11, 2024
@cfriedt cfriedt removed their assignment Jun 11, 2024
@cfriedt cfriedt changed the title posix: implement setvbuf() libc: implement setvbuf() Jun 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stephanosio stephanosio

Labels
area: C Library C Standard Library Enhancement Changes/Updates/Additions to existing features Good first issue Good for a first time contributor to take
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@nashif @cfriedt @ycsin @stephanosio @whythefyouflying