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.

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

Added line to include test requirement #1042

Closed
wants to merge 1 commit into from
Closed

Conversation

mattshu
Copy link
Contributor

@mattshu mattshu commented Mar 2, 2025

I'm still learning C (and PRs) so my skills might not be there yet — but I stumbled hard on the very first test (test_reading_empty_buffer_fails()) because I didn't realize I had to set errno to a special value.

I thought this change could serve as a small hint for newbies like me to consider how errors are raised in C. I might be overthinking it, though!

@ryanplusplus
Copy link
Member

Hey @mattshu, these docs are generated from the upstream/common problem specifications. To add track specific data we need to either add an append. Here's an example of an append on this track:

The [`time.h`][time.h] library gives you two functions to do this.

@mattshu
Copy link
Contributor Author

mattshu commented Mar 2, 2025

Thank you for the info @ryanplusplus!

@mattshu mattshu closed this Mar 2, 2025
@mattshu mattshu deleted the patch-3 branch March 2, 2025 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants