If you have enums handled in a switch adding a default case
makes it fun to fix when new stuff is added later. Remove.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
"%s [BGP_GR] GLOBAL_INVALID",
__func__);
return BGP_ERR_GR_OPERATION_FAILED;
- default:
- zlog_debug(
- "%s [BGP_GR] Global unknown ERROR",
- __func__);
- return BGP_ERR_GR_OPERATION_FAILED;
}
}
}
vty_out(vty, "\n%s", SHOW_GR_HEADER);
- int bgp_global_gr_mode = bgp_global_gr_mode_get(bgp);
+ enum global_mode bgp_global_gr_mode = bgp_global_gr_mode_get(bgp);
switch (bgp_global_gr_mode) {
break;
case GLOBAL_INVALID:
- default:
vty_out(vty,
"Global BGP GR Mode Invalid\n");
break;