mirror of
https://github.com/postgres/postgres.git
synced 2026-01-26 09:41:40 +03:00
Remove redundant pg_unreachable() after elog(ERROR) from ExecWaitStmt()
elog(ERROR) never returns. Compilers don't always understand this. So, sometimes, we have to append pg_unreachable() to keep the compiler quiet about returning from a non-void function without a value. But pg_unreachable() is redundant for ExecWaitStmt(), which is void. Reported-by: Peter Eisentraut <peter@eisentraut.org> Author: Xuneng Zhou <xunengzhou@gmail.com> Discussion: https://postgr.es/m/8d72a2b3-7423-4a15-a981-e130bf60b1a6%40eisentraut.org Discussion: https://postgr.es/m/CABPTF7UcuVD0L-X%3DjZFfeygjPaZWWkVRwtWOaJw2tcXbEN2xsA%40mail.gmail.com
This commit is contained in:
@@ -236,7 +236,6 @@ ExecWaitStmt(ParseState *pstate, WaitStmt *stmt, DestReceiver *dest)
|
||||
|
||||
default:
|
||||
elog(ERROR, "unexpected wait LSN type %d", lsnType);
|
||||
pg_unreachable();
|
||||
}
|
||||
}
|
||||
else
|
||||
@@ -281,7 +280,6 @@ ExecWaitStmt(ParseState *pstate, WaitStmt *stmt, DestReceiver *dest)
|
||||
|
||||
default:
|
||||
elog(ERROR, "unexpected wait LSN type %d", lsnType);
|
||||
pg_unreachable();
|
||||
}
|
||||
}
|
||||
else
|
||||
@@ -311,7 +309,6 @@ ExecWaitStmt(ParseState *pstate, WaitStmt *stmt, DestReceiver *dest)
|
||||
|
||||
default:
|
||||
elog(ERROR, "unexpected wait LSN type %d", lsnType);
|
||||
pg_unreachable();
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user