diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2018-10-09 13:36:16 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2018-10-09 13:36:24 -0400 |
| commit | b6b297d20df9f738be20a450f80bade535819220 (patch) | |
| tree | 3dfaf336eaa129b08662338bd50b27954c22a55a /src/backend/commands/variable.c | |
| parent | 36e9d413a1d6928cd809d302d495fd6880a44b1e (diff) | |
| download | postgresql-b6b297d20df9f738be20a450f80bade535819220.tar.gz postgresql-b6b297d20df9f738be20a450f80bade535819220.zip | |
Make src/common/exec.c's error logging less ugly.
This code used elog where it really ought to use ereport, mainly so that
it can report a SQLSTATE different from ERRCODE_INTERNAL_ERROR. There
were some other random deviations from typical error report practice too.
In addition, we can make some cleanups that were impractical six months
ago:
* Use one variadic macro, instead of several with different numbers
of arguments, reducing the temptation to force-fit messages into
particular numbers of arguments;
* Use %m, even in the frontend case, simplifying the code.
Discussion: https://postgr.es/m/6025.1527351693@sss.pgh.pa.us
Diffstat (limited to 'src/backend/commands/variable.c')
0 files changed, 0 insertions, 0 deletions