Skip to content

Conversation

@LinuxJedi
Copy link
Member

Description

Fixes API header mismatches between API and documentation.

@devin-ai-integration
Copy link
Contributor

🛟 Devin Lifeguard found 2 likely issues in this PR

  • do-not-change-external-apis snippet snippet snippet: Restore the original prototypes (e.g., keep int wc_AesFree(Aes* aes)), and introduce new versions such as wc_AesFree_ex(Aes* aes) or similar wrappers with the updated return type/parameters, having the old functions call the new ones with default values.
  • no-void-functions snippet: Revert wc_AesFree (and any other newly-changed APIs) to return an int and ensure the implementation returns an appropriate error code so callers can detect failures.

@LinuxJedi
please take a look at the above issues which Devin flagged. Devin will not fix these issues automatically.

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.

1 participant