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.
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
Documentation typo fix in EVP_EncryptInit.pod #5792
Conversation
doc/man3/EVP_EncryptInit.pod
Outdated
@@ -100,7 +100,6 @@ EVP_enc_null | |||
int EVP_CIPHER_nid(const EVP_CIPHER *e); | |||
int EVP_CIPHER_block_size(const EVP_CIPHER *e); | |||
int EVP_CIPHER_key_length(const EVP_CIPHER *e) | |||
int EVP_CIPHER_key_length(const EVP_CIPHER *e); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably it would be better to remove the one without the ; on the end!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed thanks!
Please update your commit comment with the text "CLA: Trivial" on a separate line. Assuming you agree that this change is trivial from a copyright perspective and therefore you don't need to sign the CLA. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You will either have to sign our CLA or edit your commit text to put "CLA: trivial" in the message body. I agree this is trivial.
Remove duplicate declaration of `EVP_CIPHER_key_length` in the synopsis. CLA: trivial
Merged to master, thanks for the fix! |
Remove duplicate declaration of `EVP_CIPHER_key_length` in the synopsis. CLA: trivial Reviewed-by: Andy Polyakov <appro@openssl.org> Reviewed-by: Rich Salz <rsalz@openssl.org> (Merged from #5792)
Remove duplicate declaration of
EVP_CIPHER_key_length
in the synopsis.Checklist
N/A