Replace GUC_UNIT_MEMORY|GUC_UNIT_TIME with GUC_UNIT.
authorMasahiko Sawada <[email protected]>
Thu, 15 Jun 2023 08:04:19 +0000 (17:04 +0900)
committerMasahiko Sawada <[email protected]>
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;
    }
 }