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.

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

Fix typo in ASYNC_WAIT_CTX_new.pod doc #2967

Conversation

InfoHunter
Copy link
Member

For the function that get the changed fds, it should be
'ASYNC_WAIT_CTX_get_changed_fds()' instead of 'ASYNC_WAIT_CTX_fds_have_changed()'.

Same thing as #2966, but for the 1.1.0 branch.

Signed-off-by: Paul Yang paulyang.inf@gmail.com

Checklist
Description of change

@openssl-machine openssl-machine added the hold: cla required The contributor needs to submit a license agreement label Mar 16, 2017
@mattcaswell
Copy link
Member

As per #2966, please update the commit message.

For the function that get the changed fds, it should be
'ASYNC_WAIT_CTX_get_changed_fds()' instead of 'ASYNC_WAIT_CTX_fds_have_changed()'.

CLA: trivial

Signed-off-by: Paul Yang <paulyang.inf@gmail.com>
@InfoHunter
Copy link
Member Author

Updated

@mattcaswell mattcaswell removed the hold: cla required The contributor needs to submit a license agreement label Mar 16, 2017
@levitte levitte added the approval: done This pull request has the required number of approvals label Mar 16, 2017
@mattcaswell
Copy link
Member

@levitte as for the master version of this can you confirm you are happy with this as "trivial"?

@levitte
Copy link
Member

levitte commented Mar 16, 2017

It is trivial

@mattcaswell
Copy link
Member

Pushed.

levitte pushed a commit that referenced this pull request Mar 16, 2017
For the function that get the changed fds, it should be
'ASYNC_WAIT_CTX_get_changed_fds()' instead of 'ASYNC_WAIT_CTX_fds_have_changed()'.

CLA: trivial

Signed-off-by: Paul Yang <paulyang.inf@gmail.com>

Reviewed-by: Richard Levitte <levitte@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from #2967)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approval: done This pull request has the required number of approvals
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants