Skip to content

docs: update API docs to reflect output-parameter getter signatures#56

Merged
ionux merged 1 commit intomainfrom
claude/update-docs-functions-z7oxe
Mar 17, 2026
Merged

docs: update API docs to reflect output-parameter getter signatures#56
ionux merged 1 commit intomainfrom
claude/update-docs-functions-z7oxe

Conversation

@ionux
Copy link
Copy Markdown
Owner

@ionux ionux commented Mar 17, 2026

All getter functions (okj_get_string, okj_get_number, okj_get_boolean, okj_get_array, okj_get_object, okj_get_token, okj_get_array_raw, okj_get_object_raw) were refactored from returning typed pointers to returning OkjError with a caller-supplied output struct parameter.

Update README.md, wiki/API-Reference.md, and wiki/Usage-Guide.md to:

  • Show correct three-argument getter signatures with output params
  • Replace NULL-return semantics with OkjError return code checks
  • Fix code examples to declare structs and pass their addresses
  • Remove OKJ_ERROR_TRAILING_CONTENT (not present in current OkjError enum)
  • Clarify error codes returned on size-limit violations for array/object getters

https://claude.ai/code/session_01RqYVgHBmL952SnsPn57Qf9

All getter functions (okj_get_string, okj_get_number, okj_get_boolean,
okj_get_array, okj_get_object, okj_get_token, okj_get_array_raw,
okj_get_object_raw) were refactored from returning typed pointers to
returning OkjError with a caller-supplied output struct parameter.

Update README.md, wiki/API-Reference.md, and wiki/Usage-Guide.md to:
- Show correct three-argument getter signatures with output params
- Replace NULL-return semantics with OkjError return code checks
- Fix code examples to declare structs and pass their addresses
- Remove OKJ_ERROR_TRAILING_CONTENT (not present in current OkjError enum)
- Clarify error codes returned on size-limit violations for array/object getters

https://claude.ai/code/session_01RqYVgHBmL952SnsPn57Qf9
@ionux ionux merged commit a7a165e into main Mar 17, 2026
2 checks passed
@ionux ionux deleted the claude/update-docs-functions-z7oxe branch March 17, 2026 04:56
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.

2 participants