Replace GUC_UNIT_MEMORY|GUC_UNIT_TIME with GUC_UNIT.
authorMasahiko Sawada <msawada@postgresql.org>
Thu, 15 Jun 2023 08:04:19 +0000 (17:04 +0900)
committerMasahiko Sawada <msawada@postgresql.org>
Thu, 15 Jun 2023 08:04:19 +0000 (17:04 +0900)
We used (GUC_UNIT_MEMORY | GUC_UNIT_TIME) instead of GUC_UNIT some
places but we already define it in guc.h. This commit replaces them
with GUC_UNIT for better consistency with their surrounding code.

Author: Japin Li
Reviewed-by: Richard Guo, Michael Paquier, Masahiko Sawada
Discussion: https://postgr.es/m/MEYP282MB1669EC0FED922F7A151673ACB65AA@MEYP282MB1669.AUSP282.PROD.OUTLOOK.COM

src/backend/utils/misc/guc.c

index a9033b7a54d427f54ac41fbda6ab23a7775b9b1d..5308896c87fbd7b92c4ae9f53fc343b3ff093469 100644 (file)
@@ -2766,7 +2766,7 @@ convert_real_from_base_unit(double base_value, int base_unit,
 const char *
 get_config_unit_name(int flags)
 {
-       switch (flags & (GUC_UNIT_MEMORY | GUC_UNIT_TIME))
+       switch (flags & GUC_UNIT)
        {
                case 0:
                        return NULL;            /* GUC has no units */
@@ -2802,7 +2802,7 @@ get_config_unit_name(int flags)
                        return "min";
                default:
                        elog(ERROR, "unrecognized GUC units value: %d",
-                                flags & (GUC_UNIT_MEMORY | GUC_UNIT_TIME));
+                                flags & GUC_UNIT);
                        return NULL;
        }
 }