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

Update confusing example of using CURLINFO_TLS_SSL_PTR #2690

Closed
wants to merge 2 commits into from

Conversation

@AdrianPeniak
Copy link
Contributor

@AdrianPeniak AdrianPeniak commented Jun 28, 2018

Previous example was little bit confusing, because SSL* structure (or other "in use" SSL connection pointer) are not accessible after transfer, therefore working with raw TLS library specific pointer should be done during transfer.

Previous example was little bit confusing, because SSL* structure (or other "in use" SSL connection pointer) are not accessible after transfer, therefore working with raw TLS library specific pointer should be done during transfer.
@AdrianPeniak AdrianPeniak changed the title Update confusing example of using Update confusing example of using CURLINFO_TLS_SSL_PTR Jun 28, 2018
@bagder
bagder approved these changes Jun 28, 2018
Copy link
Member

@bagder bagder left a comment

Yeah, this is better. Two nits:

  1. it mixes code-style (brace position)
  2. I think we can skip the (void) lines to shorten it
Unify brackets & remove unused (void) lines
@bagder
Copy link
Member

@bagder bagder commented Jun 28, 2018

Thanks! (I edited it slightly before merge)

@bagder bagder closed this in 24cb114 Jun 28, 2018
@lock lock bot locked as resolved and limited conversation to collaborators Sep 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.