From 1fada5d81e6769ded832a4ca62ee9371bac3fb9f Mon Sep 17 00:00:00 2001 From: Tom Lane Date: Tue, 21 Dec 2021 12:12:24 -0500 Subject: 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 --- src/test/modules/delay_execution/delay_execution.c | 2 ++ 1 file changed, 2 insertions(+) (limited to 'src/test/modules/delay_execution/delay_execution.c') diff --git a/src/test/modules/delay_execution/delay_execution.c b/src/test/modules/delay_execution/delay_execution.c index b3d0841ba80..8ec623ac527 100644 --- a/src/test/modules/delay_execution/delay_execution.c +++ b/src/test/modules/delay_execution/delay_execution.c @@ -91,6 +91,8 @@ _PG_init(void) NULL, NULL); + EmitWarningsOnPlaceholders("delay_execution"); + /* Install our hook */ prev_planner_hook = planner_hook; planner_hook = delay_execution_planner; -- cgit v1.2.3