diff options
| author |  Jan Tattermusch <jtattermusch@users.noreply.github.com> | 2016-11-16 18:21:42 +0100 |
|---|---|---|
| committer |  GitHub <noreply@github.com> | 2016-11-16 18:21:42 +0100 |
| commit | c280b568b37f91c6622abe090dac6fb6f2717fbf (patch) | |
| tree | 7158e65affa3caa18dec9074386591084f6a0340 | |
| parent | 7647b2eb0a02cdf7221e5487ac1d9510aea8462e (diff) | |
| parent | b0fb2d2b7a96a408ecde0e2c1e77511d7ca88677 (diff) | |
Merge pull request #8765 from jtattermusch/fix_unreacheable_code_usage
Fix usage of GPR_UNREACHABLE_CODE
| -rw-r--r-- | src/core/ext/lb_policy/pick_first/pick_first.c | 2 | ||||
| -rw-r--r-- | src/core/ext/lb_policy/round_robin/round_robin.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/core/ext/lb_policy/pick_first/pick_first.c b/src/core/ext/lb_policy/pick_first/pick_first.c index 39f3b3d60f..c69f773e78 100644 --- a/src/core/ext/lb_policy/pick_first/pick_first.c +++ b/src/core/ext/lb_policy/pick_first/pick_first.c @@ -293,7 +293,7 @@ static void pf_connectivity_changed(grpc_exec_ctx *exec_ctx, void *arg, loop: switch (p->checking_connectivity) { case GRPC_CHANNEL_INIT: - GPR_UNREACHABLE_CODE(); + GPR_UNREACHABLE_CODE(return ); case GRPC_CHANNEL_READY: grpc_connectivity_state_set(exec_ctx, &p->state_tracker, GRPC_CHANNEL_READY, GRPC_ERROR_NONE, diff --git a/src/core/ext/lb_policy/round_robin/round_robin.c b/src/core/ext/lb_policy/round_robin/round_robin.c index acb2f77e45..59f84054c4 100644 --- a/src/core/ext/lb_policy/round_robin/round_robin.c +++ b/src/core/ext/lb_policy/round_robin/round_robin.c @@ -553,7 +553,7 @@ static void rr_connectivity_changed(grpc_exec_ctx *exec_ctx, void *arg, } switch (sd->curr_connectivity_state) { case GRPC_CHANNEL_INIT: - GPR_UNREACHABLE_CODE(); + GPR_UNREACHABLE_CODE(return ); case GRPC_CHANNEL_READY: /* add the newly connected subchannel to the list of connected ones. * Note that it goes to the "end of the line". */ |