diff options
author | ncteisen <ncteisen@gmail.com> | 2018-07-17 11:26:55 -0700 |
---|---|---|
committer | ncteisen <ncteisen@gmail.com> | 2018-07-17 11:26:55 -0700 |
commit | 0f6e4dd20d3e79d3465a7bf8a21d704e7b67b102 (patch) | |
tree | 697af2a691c6a0c03d05e6b239cad5a99766224c /src/core/ext/filters/client_channel/subchannel.cc | |
parent | adfa81987af4b61eb11c92c6e4bedc3bed3028c9 (diff) |
reviewer feedback:
Diffstat (limited to 'src/core/ext/filters/client_channel/subchannel.cc')
-rw-r--r-- | src/core/ext/filters/client_channel/subchannel.cc | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/src/core/ext/filters/client_channel/subchannel.cc b/src/core/ext/filters/client_channel/subchannel.cc index 86026b70a5..9d608c3c55 100644 --- a/src/core/ext/filters/client_channel/subchannel.cc +++ b/src/core/ext/filters/client_channel/subchannel.cc @@ -181,9 +181,7 @@ static void connection_destroy(void* arg, grpc_error* error) { static void subchannel_destroy(void* arg, grpc_error* error) { grpc_subchannel* c = static_cast<grpc_subchannel*>(arg); - if (c->channelz_subchannel != nullptr) { - c->channelz_subchannel.reset(); - } + c->channelz_subchannel.reset(); gpr_free((void*)c->filters); grpc_channel_args_destroy(c->args); grpc_connectivity_state_destroy(&c->state_tracker); @@ -380,7 +378,6 @@ grpc_subchannel* grpc_subchannel_create(grpc_connector* connector, c->backoff.Init(backoff_options); gpr_mu_init(&c->mu); - // This is just a placeholder channelz class for for now. const grpc_arg* arg = grpc_channel_args_find(c->args, GRPC_ARG_ENABLE_CHANNELZ); bool channelz_enabled = grpc_channel_arg_get_bool(arg, false); |