diff options
| author |  Mark D. Roth <roth@google.com> | 2018-12-07 12:24:18 -0800 |
|---|---|---|
| committer |  GitHub <noreply@github.com> | 2018-12-07 12:24:18 -0800 |
| commit | 680a3546815d564fca8f2abe0cae441e78c25798 (patch) | |
| tree | e625fcc6d3bf473cf020cd9fb26e248a879530a8 | |
| parent | 09be5d095f0dc5bb797d322ce0262a9be45eb157 (diff) | |
| parent | c5528b821b80f14f49d3a2857b12aecad0c6f009 (diff) | |
Merge pull request #17442 from markdroth/client_channel_pick_init
Remove unnecessary initialization of fields in PickState.
| -rw-r--r-- | src/core/ext/filters/client_channel/client_channel.cc | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/src/core/ext/filters/client_channel/client_channel.cc b/src/core/ext/filters/client_channel/client_channel.cc index 3347676a48..ebc412b468 100644 --- a/src/core/ext/filters/client_channel/client_channel.cc +++ b/src/core/ext/filters/client_channel/client_channel.cc @@ -570,12 +570,6 @@ static void start_transport_op_locked(void* arg, grpc_error* error_ignored) { } else { grpc_error* error = GRPC_ERROR_NONE; grpc_core::LoadBalancingPolicy::PickState pick_state; - pick_state.initial_metadata = nullptr; - pick_state.initial_metadata_flags = 0; - pick_state.on_complete = nullptr; - memset(&pick_state.subchannel_call_context, 0, - sizeof(pick_state.subchannel_call_context)); - pick_state.user_data = nullptr; // Pick must return synchronously, because pick_state.on_complete is null. GPR_ASSERT(chand->lb_policy->PickLocked(&pick_state, &error)); if (pick_state.connected_subchannel != nullptr) { |