diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2021-12-21 12:12:24 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2021-12-21 12:12:24 -0500 |
| commit | 1fada5d81e6769ded832a4ca62ee9371bac3fb9f (patch) | |
| tree | b645a3f7fc4340bcc0e6596309231c356a216d86 /src/backend/utils/misc/guc.c | |
| parent | 0e6e7f0806b2080cb31f33ff992ec2e4e35fa6f1 (diff) | |
| download | postgresql-1fada5d81e6769ded832a4ca62ee9371bac3fb9f.tar.gz postgresql-1fada5d81e6769ded832a4ca62ee9371bac3fb9f.zip | |
Add missing EmitWarningsOnPlaceholders() calls.
Extensions that define any custom GUCs should call
EmitWarningsOnPlaceholders after doing so, to help catch misspellings.
Many of our contrib modules hadn't gotten the memo on that, though.
Also add such calls to src/test/modules extensions that have GUCs.
While these aren't really user-facing, they should illustrate good
practice not faulty practice.
Shinya Kato
Discussion: https://postgr.es/m/524fa2c0a34f34b68fbfa90d0760d515@oss.nttdata.com
Diffstat (limited to 'src/backend/utils/misc/guc.c')
| -rw-r--r-- | src/backend/utils/misc/guc.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/backend/utils/misc/guc.c b/src/backend/utils/misc/guc.c index 7b030463013..bff949a40bc 100644 --- a/src/backend/utils/misc/guc.c +++ b/src/backend/utils/misc/guc.c @@ -9194,6 +9194,9 @@ reapply_stacked_values(struct config_generic *variable, } } +/* + * Functions for extensions to call to define their custom GUC variables. + */ void DefineCustomBoolVariable(const char *name, const char *short_desc, @@ -9333,6 +9336,10 @@ DefineCustomEnumVariable(const char *name, define_custom_variable(&var->gen); } +/* + * Extensions should call this after they've defined all of their custom + * GUCs, to help catch misspelled config-file entries, + */ void EmitWarningsOnPlaceholders(const char *className) { |