diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2020-01-30 13:32:04 -0300 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2020-01-30 13:32:04 -0300 |
| commit | 4e89c79a52f8a898edd648b56a00f0f4f840cfe7 (patch) | |
| tree | ba259edb0559281eaf0953d6995781b264a1aec7 /src/backend/utils/adt/genfile.c | |
| parent | 2520cf8c2a0a1d679094dffbd99871884e620ed5 (diff) | |
| download | postgresql-4e89c79a52f8a898edd648b56a00f0f4f840cfe7.tar.gz postgresql-4e89c79a52f8a898edd648b56a00f0f4f840cfe7.zip | |
Remove excess parens in ereport() calls
Cosmetic cleanup, not worth backpatching.
Discussion: https://postgr.es/m/20200129200401.GA6303@alvherre.pgsql
Reviewed-by: Tom Lane, Michael Paquier
Diffstat (limited to 'src/backend/utils/adt/genfile.c')
| -rw-r--r-- | src/backend/utils/adt/genfile.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/backend/utils/adt/genfile.c b/src/backend/utils/adt/genfile.c index 0d75928e7f3..3741b87486d 100644 --- a/src/backend/utils/adt/genfile.c +++ b/src/backend/utils/adt/genfile.c @@ -78,7 +78,7 @@ convert_and_check_filename(text *arg) if (path_contains_parent_reference(filename)) ereport(ERROR, (errcode(ERRCODE_INSUFFICIENT_PRIVILEGE), - (errmsg("reference to parent directory (\"..\") not allowed")))); + errmsg("reference to parent directory (\"..\") not allowed"))); /* * Allow absolute paths if within DataDir or Log_directory, even @@ -89,12 +89,12 @@ convert_and_check_filename(text *arg) !path_is_prefix_of_path(Log_directory, filename))) ereport(ERROR, (errcode(ERRCODE_INSUFFICIENT_PRIVILEGE), - (errmsg("absolute path not allowed")))); + errmsg("absolute path not allowed"))); } else if (!path_is_relative_and_below_cwd(filename)) ereport(ERROR, (errcode(ERRCODE_INSUFFICIENT_PRIVILEGE), - (errmsg("path must be in or below the current directory")))); + errmsg("path must be in or below the current directory"))); return filename; } @@ -218,10 +218,10 @@ pg_read_file(PG_FUNCTION_ARGS) if (!superuser()) ereport(ERROR, (errcode(ERRCODE_INSUFFICIENT_PRIVILEGE), - (errmsg("must be superuser to read files with adminpack 1.0"), + errmsg("must be superuser to read files with adminpack 1.0"), /* translator: %s is a SQL function name */ - errhint("Consider using %s, which is part of core, instead.", - "pg_file_read()")))); + errhint("Consider using %s, which is part of core, instead.", + "pg_file_read()"))); /* handle optional arguments */ if (PG_NARGS() >= 3) |