Code review changes.

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);

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;