Skip to content

docs: Added comments for 3 private functions (fixes #371)#546

Merged
goatshriek merged 1 commit intogoatshriek:latestfrom
juliafeldhousen:latest
Nov 14, 2025
Merged

docs: Added comments for 3 private functions (fixes #371)#546
goatshriek merged 1 commit intogoatshriek:latestfrom
juliafeldhousen:latest

Conversation

@juliafeldhousen
Copy link
Contributor

Added documentation for three undocumented private functions: lock_param() in param.h, cap_size_t_to_int() in inthelper.h, and copy_cstring() in strhelper.h. This is a part of ongoing work on issue #371. Let me know if any changes are needed!

@goatshriek goatshriek self-assigned this Nov 13, 2025
@goatshriek goatshriek added the documentation documentation updates label Nov 13, 2025
@codecov
Copy link

codecov bot commented Nov 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.64%. Comparing base (d843cb0) to head (bfc2161).
⚠️ Report is 1 commits behind head on latest.

Additional details and impacted files
@@            Coverage Diff             @@
##           latest     #546      +/-   ##
==========================================
- Coverage   90.82%   90.64%   -0.18%     
==========================================
  Files          47       47              
  Lines        4566     4566              
  Branches      605      605              
==========================================
- Hits         4147     4139       -8     
- Misses        276      283       +7     
- Partials      143      144       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@goatshriek goatshriek merged commit 1ad39fe into goatshriek:latest Nov 14, 2025
55 of 56 checks passed
@goatshriek
Copy link
Owner

Thanks very much for putting this together!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation documentation updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants