diff options
| author | Peter Eisentraut <peter@eisentraut.org> | 2019-11-01 11:09:52 +0100 |
|---|---|---|
| committer | Peter Eisentraut <peter@eisentraut.org> | 2019-11-01 11:18:03 +0100 |
| commit | 604bd3671121b51f977de146ed95484c2297fb3e (patch) | |
| tree | 829045453c054befb9d795c33765501e2f4522f3 /src/backend/commands/copy.c | |
| parent | 73025140885c889410b9bfc4a30a3866396fc5db (diff) | |
| download | postgresql-604bd3671121b51f977de146ed95484c2297fb3e.tar.gz postgresql-604bd3671121b51f977de146ed95484c2297fb3e.zip | |
PG_FINALLY
This gives an alternative way of catching exceptions, for the common
case where the cleanup code is the same in the error and non-error
cases. So instead of
PG_TRY();
{
... code that might throw ereport(ERROR) ...
}
PG_CATCH();
{
cleanup();
PG_RE_THROW();
}
PG_END_TRY();
cleanup();
one can write
PG_TRY();
{
... code that might throw ereport(ERROR) ...
}
PG_FINALLY();
{
cleanup();
}
PG_END_TRY();
Discussion: https://www.postgresql.org/message-id/flat/95a822c3-728b-af0e-d7e5-71890507ae0c%402ndquadrant.com
Diffstat (limited to 'src/backend/commands/copy.c')
| -rw-r--r-- | src/backend/commands/copy.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/backend/commands/copy.c b/src/backend/commands/copy.c index 3aeef30b281..e17d8c760f7 100644 --- a/src/backend/commands/copy.c +++ b/src/backend/commands/copy.c @@ -1916,13 +1916,11 @@ BeginCopyTo(ParseState *pstate, { cstate->copy_file = AllocateFile(cstate->filename, PG_BINARY_W); } - PG_CATCH(); + PG_FINALLY(); { umask(oumask); - PG_RE_THROW(); } PG_END_TRY(); - umask(oumask); if (cstate->copy_file == NULL) { /* copy errno because ereport subfunctions might change it */ |