Fix implicit-fallthrough compiler warning introduced in 6dda292.

anarazel · anarazel · commit c967e13f4047 · 2019-09-27T10:29:25.000-07:00
For some reason at least gcc-9 warns about the fallthrough, even
though it otherwise recognizes that elog(ERROR, ...) doesn't return.

Author: Andres Freund
diff --git a/src/backend/utils/adt/jsonb_util.c b/src/backend/utils/adt/jsonb_util.c
@@ -247,8 +247,10 @@ compareJsonbContainers(JsonbContainer *a, JsonbContainer *b)
 						break;
 					case jbvBinary:
 						elog(ERROR, "unexpected jbvBinary value");
+						break;
 					case jbvDatetime:
 						elog(ERROR, "unexpected jbvDatetime value");
+						break;
 				}
 			}
 			else

Original file line number	Diff line number	Diff line change
`@@ -247,8 +247,10 @@ compareJsonbContainers(JsonbContainer a, JsonbContainer b)`
`247`	`247`	`break;`
`248`	`248`	`case jbvBinary:`
`249`	`249`	`elog(ERROR, "unexpected jbvBinary value");`
	`250`	`+ break;`
`250`	`251`	`case jbvDatetime:`
`251`	`252`	`elog(ERROR, "unexpected jbvDatetime value");`
	`253`	`+ break;`
`252`	`254`	`}`
`253`	`255`	`}`
`254`	`256`	`else`