Skip to content

hash: provide asserts to verify API use - #14503

Closed
bagder wants to merge 3 commits into
masterfrom
bagder/hash-asserts
Closed

bagder wants to merge 3 commits into
masterfrom
bagder/hash-asserts

Conversation

@bagder

@bagder bagder commented Aug 12, 2024 •

Copy link
Copy Markdown
Member
  • converted the Curl_hash_count() macro to a function
  • Discourage accessing struct fields directly
  • Document the API

@bagder bagder added the tidy-up label Aug 12, 2024
@github-actions github-actions Bot added the tests label Aug 12, 2024
@bagder
bagder force-pushed the bagder/hash-asserts branch from 7e4a0ec to ddabd73 Compare August 12, 2024 13:52
@bagder

This comment was marked as resolved.

@bagder
bagder force-pushed the bagder/hash-asserts branch 2 times, most recently from 5dc87b2 to 270450f Compare August 12, 2024 22:09
- converted the Curl_hash_count() macro to a function

- Discourage accessing struct fields directly

- Document the internal API in HASH.md

Closes #14503
@bagder
bagder force-pushed the bagder/hash-asserts branch from 6f55d10 to 7ea1698 Compare August 13, 2024 12:27
@bagder bagder closed this in c0233a3 Aug 15, 2024
@bagder
bagder deleted the bagder/hash-asserts branch August 15, 2024 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant