diff options
author | Mark D. Roth <roth@google.com> | 2016-09-16 13:03:46 -0700 |
---|---|---|
committer | Mark D. Roth <roth@google.com> | 2016-09-16 13:03:46 -0700 |
commit | 7ce14d2a008d0df70ec0a43235728f4d5781ee36 (patch) | |
tree | 89e47137992946e8e4ddb4b2e558202ee41afcff /src | |
parent | 2b626466af11eed576859499baecf8c807591912 (diff) |
Code review changes.
Diffstat (limited to 'src')
-rw-r--r-- | src/core/ext/client_config/lb_policy_factory.c | 3 | ||||
-rw-r--r-- | src/core/ext/lb_policy/grpclb/grpclb.c | 3 |
2 files changed, 3 insertions, 3 deletions
diff --git a/src/core/ext/client_config/lb_policy_factory.c b/src/core/ext/client_config/lb_policy_factory.c index e5e416ced9..f86117aa38 100644 --- a/src/core/ext/client_config/lb_policy_factory.c +++ b/src/core/ext/client_config/lb_policy_factory.c @@ -63,8 +63,9 @@ void grpc_lb_addresses_destroy(grpc_lb_addresses* addresses, void (*user_data_destroy)(void*)) { for (size_t i = 0; i < addresses->num_addresses; ++i) { gpr_free(addresses->addresses[i].balancer_name); - if (user_data_destroy != NULL) + if (user_data_destroy != NULL) { user_data_destroy(addresses->addresses[i].user_data); + } } gpr_free(addresses->addresses); gpr_free(addresses); diff --git a/src/core/ext/lb_policy/grpclb/grpclb.c b/src/core/ext/lb_policy/grpclb/grpclb.c index 999122fbe0..36db8ab00d 100644 --- a/src/core/ext/lb_policy/grpclb/grpclb.c +++ b/src/core/ext/lb_policy/grpclb/grpclb.c @@ -354,8 +354,7 @@ static bool is_server_valid(const grpc_grpclb_server *server, size_t idx, return true; } -/* populate \a addresses according to \a serverlist. Returns the number of - * addresses successfully parsed and added to \a addresses */ +/* Returns addresses extracted from \a serverlist. */ static grpc_lb_addresses *process_serverlist( const grpc_grpclb_serverlist *serverlist) { size_t num_valid = 0; |