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.

Sign up for GitHub

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

Jump to bottom

tls: fix some doxygen warnings #894

Merged
merged 2 commits into from
Aug 9, 2023
Merged

tls: fix some doxygen warnings #894

merged 2 commits into from
Aug 9, 2023

Conversation

alfredh
Copy link
Contributor

@alfredh alfredh commented Aug 9, 2023 •
edited
Loading 

/home/alfredh/git/re/src/tls/openssl/tls.c:919: warning: argument 'pem' of command @param is not found in the argument list of tls_set_certificate(struct tls *tls, const char *cert, size_t len)
/home/alfredh/git/re/src/tls/openssl/tls.c:919: warning: The following parameter of tls_set_certificate(struct tls *tls, const char *cert, size_t len) is not documented:
  parameter 'cert'
/home/alfredh/git/re/src/tls/openssl/tls.c:2100: warning: argument 'value' of command @param is not found in the argument list of tls_set_posthandshake_auth(struct tls *tls, int enabled)
/home/alfredh/git/re/src/tls/openssl/tls.c:2100: warning: The following parameter of tls_set_posthandshake_auth(struct tls *tls, int enabled) is not documented:
  parameter 'enabled'

@alfredh alfredh added this to the v3.5.0 milestone Aug 9, 2023
@sreimers sreimers merged commit b788cf4 into main Aug 9, 2023
38 checks passed
@sreimers sreimers deleted the alfredh_tls_doxygen branch August 9, 2023 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v3.5.0
Development

Successfully merging this pull request may close these issues.
2 participants
@alfredh @sreimers