aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/core/lib/iomgr/tcp_client_cfstream.cc
diff options
context:
space:
mode:
authorGravatar Muxi Yan <mxyan@google.com>2018-06-01 10:26:35 -0700
committerGravatar Muxi Yan <mxyan@google.com>2018-06-01 10:26:35 -0700
commit72f7159546fb1e993151935f6eef1eaade2c9126 (patch)
tree64f0ac4945c53b6c7ae89d50b96b938dd42f34ac /src/core/lib/iomgr/tcp_client_cfstream.cc
parent2111cc0a8e2b5c11a9932fd440f78405b9d3ad08 (diff)
Polish comment
Diffstat (limited to 'src/core/lib/iomgr/tcp_client_cfstream.cc')
-rw-r--r--src/core/lib/iomgr/tcp_client_cfstream.cc4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/core/lib/iomgr/tcp_client_cfstream.cc b/src/core/lib/iomgr/tcp_client_cfstream.cc
index 87e5bab567..b1a40f7a30 100644
--- a/src/core/lib/iomgr/tcp_client_cfstream.cc
+++ b/src/core/lib/iomgr/tcp_client_cfstream.cc
@@ -89,7 +89,7 @@ static void OnAlarm(void* arg, grpc_error* error) {
connect->closure = nil;
const bool done = (--connect->refs == 0);
gpr_mu_unlock(&connect->mu);
- // Only schedule a callback once, by either on_timer or on_connected. The
+ // Only schedule a callback once, by either OnAlarm or OnOpen. The
// first one issues callback while the second one does cleanup.
if (done) {
CFStreamConnectCleanup(connect);
@@ -113,6 +113,8 @@ static void OnOpen(void* arg, grpc_error* error) {
bool done = (--connect->refs == 0);
grpc_endpoint** endpoint = connect->endpoint;
+ // Only schedule a callback once, by either OnAlarm or OnOpen. The
+ // first one issues callback while the second one does cleanup.
if (done) {
gpr_mu_unlock(&connect->mu);
CFStreamConnectCleanup(connect);