Skip to content

Address comments from #8913. Also make changes needed for import.#8979

Merged
markdroth merged 13 commits intogrpc:masterfrom
markdroth:handshaker_cleanup
Dec 9, 2016
Merged

Address comments from #8913. Also make changes needed for import.#8979
markdroth merged 13 commits intogrpc:masterfrom
markdroth:handshaker_cleanup

Conversation

@markdroth
Copy link
Copy Markdown
Member

@markdroth markdroth commented Dec 6, 2016

This is the last part of fixing #8744.

@markdroth markdroth changed the title Address comments from #8913. Address comments from #8913. Also make changes needed for import. Dec 8, 2016
// Create handshakers.
grpc_security_create_handshakers(exec_ctx, tsi_hs, &sc->base, handshake_mgr);
grpc_handshake_manager_add(handshake_mgr, grpc_security_handshaker_create(
exec_ctx, tsi_hs, &sc->base));
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did you clang-format? Indentation looks off.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh actually, you did. Weird...

@markdroth markdroth requested a review from yang-g December 8, 2016 22:00
Copy link
Copy Markdown
Contributor

@yang-g yang-g left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@markdroth markdroth merged commit be37dff into grpc:master Dec 9, 2016
@markdroth markdroth deleted the handshaker_cleanup branch December 9, 2016 21:39
@lock lock bot locked as resolved and limited conversation to collaborators Jan 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants