Further -Wimplicit-fallthrough cleanup.

Tom's earlier commit in 41c912cad159 didn't update a few cases that
are only encountered with the non-standard --with-llvm config
flag. Additionally there's also one case that appears to be a
deficiency in gcc's (up to trunk as of a few days ago) detection of
"fallthrough" comments - changing the placement slightly fixes that.

Author: Andres Freund
Discussion: https://postgr.es/m/20180502003239.wfnqu7ekz7j7imm4@alap3.anarazel.de

diff --git a/src/backend/jit/llvm/llvmjit_expr.c b/src/backend/jit/llvm/llvmjit_expr.c
index c8ec4be85211fae4cf0cae24785f7e9b13798925..36c5f7d500e889164d60172aca02d11c8e7955b8 100644 (file)
--- a/src/backend/jit/llvm/llvmjit_expr.c
+++ b/src/backend/jit/llvm/llvmjit_expr.c
@@ -672,7 +672,8 @@ llvm_compile_expr(ExprState *state)
 
                                        LLVMPositionBuilderAtEnd(b, b_nonull);
                                }
-                               /* explicit fallthrough */
+                               /* FALLTHROUGH */
+
                        case EEOP_FUNCEXPR:
                                {
                                        FunctionCallInfo fcinfo = op->d.func.fcinfo_data;
@@ -709,8 +710,8 @@ llvm_compile_expr(ExprState *state)
                                                                                                 l_ptr(TypeStorageBool));
                                        LLVMBuildStore(b, l_sbool_const(0), v_boolanynullp);
 
-                                       /* intentionally fall through */
                                }
+                               /* FALLTHROUGH */
 
                                /*
                                 * Treat them the same for now, optimizer can remove
@@ -810,9 +811,8 @@ llvm_compile_expr(ExprState *state)
                                        v_boolanynullp = l_ptr_const(op->d.boolexpr.anynull,
                                                                                                 l_ptr(TypeStorageBool));
                                        LLVMBuildStore(b, l_sbool_const(0), v_boolanynullp);
-
-                                       /* intentionally fall through */
                                }
+                               /* FALLTHROUGH */
 
                                /*
                                 * Treat them the same for now, optimizer can remove
@@ -2083,9 +2083,8 @@ llvm_compile_expr(ExprState *state)
                                                                        opblocks[op->d.agg_deserialize.jumpnull],
                                                                        b_deserialize);
                                        LLVMPositionBuilderAtEnd(b, b_deserialize);
-
-                                       /* fallthrough */
                                }
+                               /* FALLTHROUGH */
 
                        case EEOP_AGG_DESERIALIZE:
                                {

diff --git a/src/interfaces/libpq/fe-secure.c b/src/interfaces/libpq/fe-secure.c
index cfb77f6d85f65f9f50f93236468ffb605e9117e1..f7dc249bf0c4cd4563c4ec8b1dc9a208c680271e 100644 (file)
--- a/src/interfaces/libpq/fe-secure.c
+++ b/src/interfaces/libpq/fe-secure.c
@@ -360,9 +360,10 @@ retry_masked:
                        case EPIPE:
                                /* Set flag for EPIPE */
                                REMEMBER_EPIPE(spinfo, true);
-                               /* FALL THRU */
 
 #ifdef ECONNRESET
+                               /* FALL THRU */
+
                        case ECONNRESET:
 #endif
                                printfPQExpBuffer(&conn->errorMessage,