From 80e8450a744b1f6fa75663f37f1db3388995dc67 Mon Sep 17 00:00:00 2001 From: John Naylor Date: Tue, 16 Aug 2022 12:01:41 +0700 Subject: [PATCH] Move private declarations shared between guc.c and guc-file.l to new header Further preparatory refactoring for compiling guc-file.c standalone. Reviewed by Andres Freund Discussion: https://www.postgresql.org/message-id/20220810171935.7k5zgnjwqzalzmtm%40awork3.anarazel.de Discussion: https://www.postgresql.org/message-id/CAFBsxsF8Gc2StS3haXofshHCzqNMRXiSxvQEYGwnFsTmsdwNeg@mail.gmail.com --- src/backend/utils/misc/guc-file.l | 2 +- src/backend/utils/misc/guc.c | 1 + src/backend/utils/misc/guc_internal.h | 26 ++++++++++++++++++++++++++ src/include/utils/guc.h | 10 ---------- 4 files changed, 28 insertions(+), 11 deletions(-) create mode 100644 src/backend/utils/misc/guc_internal.h diff --git a/src/backend/utils/misc/guc-file.l b/src/backend/utils/misc/guc-file.l index 47d4bd18d56..8b5b299efd8 100644 --- a/src/backend/utils/misc/guc-file.l +++ b/src/backend/utils/misc/guc-file.l @@ -15,10 +15,10 @@ #include #include "common/file_utils.h" +#include "guc_internal.h" #include "mb/pg_wchar.h" #include "miscadmin.h" #include "storage/fd.h" -#include "utils/guc.h" /* diff --git a/src/backend/utils/misc/guc.c b/src/backend/utils/misc/guc.c index 66ab3912a0e..0e6c13fa2a7 100644 --- a/src/backend/utils/misc/guc.c +++ b/src/backend/utils/misc/guc.c @@ -57,6 +57,7 @@ #include "commands/variable.h" #include "common/string.h" #include "funcapi.h" +#include "guc_internal.h" #include "jit/jit.h" #include "libpq/auth.h" #include "libpq/libpq.h" diff --git a/src/backend/utils/misc/guc_internal.h b/src/backend/utils/misc/guc_internal.h new file mode 100644 index 00000000000..be3940951a2 --- /dev/null +++ b/src/backend/utils/misc/guc_internal.h @@ -0,0 +1,26 @@ +/*-------------------------------------------------------------------- + * guc_internals.h + * + * Declarations shared between backend/utils/misc/guc.c and + * backend/utils/misc/guc-file.l + * + * Copyright (c) 2000-2022, PostgreSQL Global Development Group + * + * src/include/utils/guc_internals.h + *-------------------------------------------------------------------- + */ +#ifndef GUC_INTERNALS_H +#define GUC_INTERNALS_H + +#include "utils/guc.h" + +extern int guc_name_compare(const char *namea, const char *nameb); +extern ConfigVariable *ProcessConfigFileInternal(GucContext context, + bool applySettings, int elevel); +extern void record_config_file_error(const char *errmsg, + const char *config_file, + int lineno, + ConfigVariable **head_p, + ConfigVariable **tail_p); + +#endif /* GUC_INTERNALS_H */ diff --git a/src/include/utils/guc.h b/src/include/utils/guc.h index aae071cd825..45ae1b537f3 100644 --- a/src/include/utils/guc.h +++ b/src/include/utils/guc.h @@ -442,16 +442,6 @@ extern void GUC_check_errcode(int sqlerrcode); pre_format_elog_string(errno, TEXTDOMAIN), \ GUC_check_errhint_string = format_elog_string -/* functions shared between guc.c and guc-file.l */ -extern int guc_name_compare(const char *namea, const char *nameb); -extern ConfigVariable *ProcessConfigFileInternal(GucContext context, - bool applySettings, int elevel); -extern void record_config_file_error(const char *errmsg, - const char *config_file, - int lineno, - ConfigVariable **head_p, - ConfigVariable **tail_p); - /* * The following functions are not in guc.c, but are declared here to avoid * having to include guc.h in some widely used headers that it really doesn't