Enable SET value = DEFAULT by passing null parameter to parsers.

Enable SET TIME ZONE using TZ environment variable.

diff --git a/src/backend/tcop/variable.c b/src/backend/tcop/variable.c
index dc6300a81184afc3b47e3ebefef8c8d3304a6d60..26d4096828d650aa9579ed1d63a344b8f49b8f4e 100644 (file)
--- a/src/backend/tcop/variable.c
+++ b/src/backend/tcop/variable.c
@@ -2,7 +2,7 @@
  * Routines for handling of 'SET var TO',
  *  'SHOW var' and 'RESET var' statements.
  *
- * $Id: variable.c,v 1.18 1997/10/30 16:52:11 thomas Exp $
+ * $Id: variable.c,v 1.19 1997/11/07 06:43:16 thomas Exp $
  *
  */
 
@@ -156,13 +156,19 @@ reset_null(const char *value)
 }
 #endif
 
-static bool
+bool
 parse_geqo(const char *value)
 {
    const char *rest;
    char       *tok,
               *val;
 
+   if (value == NULL)
+   {
+       reset_geqo();
+       return TRUE;
+   }
+
    rest = get_token(&tok, &val, value);
    if (tok == NULL)
        elog(WARN, "Value undefined");
@@ -197,7 +203,7 @@ parse_geqo(const char *value)
    return TRUE;
 }
 
-static bool
+bool
 show_geqo()
 {
 
@@ -208,8 +214,8 @@ show_geqo()
    return TRUE;
 }
 
-static bool
-reset_geqo()
+bool
+reset_geqo(void)
 {
 
 #ifdef GEQO
@@ -221,9 +227,14 @@ reset_geqo()
    return TRUE;
 }
 
-static bool
+bool
 parse_r_plans(const char *value)
 {
+   if (value == NULL)
+   {
+       reset_r_plans();
+       return TRUE;
+   }
 
    if (strcasecmp(value, "on") == 0)
        _use_right_sided_plans_ = true;
@@ -235,7 +246,7 @@ parse_r_plans(const char *value)
    return TRUE;
 }
 
-static bool
+bool
 show_r_plans()
 {
 
@@ -246,7 +257,7 @@ show_r_plans()
    return TRUE;
 }
 
-static bool
+bool
 reset_r_plans()
 {
 
@@ -258,17 +269,24 @@ reset_r_plans()
    return TRUE;
 }
 
-static bool
+bool
 parse_cost_heap(const char *value)
 {
-   float32     res = float4in((char *) value);
+   float32     res;
+
+   if (value == NULL)
+   {
+       reset_cost_heap();
+       return TRUE;
+   }
 
+   res = float4in((char *) value);
    _cpu_page_wight_ = *res;
 
    return TRUE;
 }
 
-static bool
+bool
 show_cost_heap()
 {
 
@@ -276,24 +294,31 @@ show_cost_heap()
    return TRUE;
 }
 
-static bool
+bool
 reset_cost_heap()
 {
    _cpu_page_wight_ = _CPU_PAGE_WEIGHT_;
    return TRUE;
 }
 
-static bool
+bool
 parse_cost_index(const char *value)
 {
-   float32     res = float4in((char *) value);
+   float32     res;
+
+   if (value == NULL)
+   {
+       reset_cost_index();
+       return TRUE;
+   }
 
+   res = float4in((char *) value);
    _cpu_index_page_wight_ = *res;
 
    return TRUE;
 }
 
-static bool
+bool
 show_cost_index()
 {
 
@@ -301,20 +326,26 @@ show_cost_index()
    return TRUE;
 }
 
-static bool
+bool
 reset_cost_index()
 {
    _cpu_index_page_wight_ = _CPU_INDEX_PAGE_WEIGHT_;
    return TRUE;
 }
 
-static bool
+bool
 parse_date(const char *value)
 {
    char       *tok;
    int         dcnt = 0,
                ecnt = 0;
 
+   if (value == NULL)
+   {
+       reset_date();
+       return TRUE;
+   }
+
    while ((value = get_token(&tok, NULL, value)) != 0)
    {
        /* Ugh. Somebody ought to write a table driven version -- mjl */
@@ -364,7 +395,7 @@ parse_date(const char *value)
    return TRUE;
 }
 
-static bool
+bool
 show_date()
 {
    char        buf[64];
@@ -391,7 +422,7 @@ show_date()
    return TRUE;
 }
 
-static bool
+bool
 reset_date()
 {
    DateStyle = USE_POSTGRES_DATES;
@@ -400,13 +431,35 @@ reset_date()
    return TRUE;
 }
 
-static bool
+static char *defaultTZ = NULL;
+static char TZvalue[10];
+
+bool
 parse_timezone(const char *value)
 {
    char       *tok;
 
+   if (value == NULL)
+   {
+       reset_timezone();
+       return TRUE;
+   }
+
    while ((value = get_token(&tok, NULL, value)) != 0)
    {
+       if ((defaultTZ == NULL) && (getenv("TZ") != NULL))
+       {
+           defaultTZ = getenv("TZ");
+           if (defaultTZ == NULL)
+           {
+               defaultTZ = (char *) -1;
+           }
+           else
+           {
+               strcpy(TZvalue, defaultTZ);
+           }
+       }
+
        setenv("TZ", tok, TRUE);
        tzset();
        PFREE(tok);
@@ -415,7 +468,7 @@ parse_timezone(const char *value)
    return TRUE;
 } /* parse_timezone() */
 
-static bool
+bool
 show_timezone()
 {
    char        buf[64];
@@ -431,10 +484,17 @@ show_timezone()
    return TRUE;
 } /* show_timezone() */
 
-static bool
+bool
 reset_timezone()
 {
-   unsetenv("TZ");
+   if ((defaultTZ != NULL) && (defaultTZ != (char *) -1))
+   {
+       setenv("TZ", TZvalue, TRUE);
+   }
+   else
+   {
+       unsetenv("TZ");
+   }
    tzset();
 
    return TRUE;
@@ -457,12 +517,10 @@ struct VariableParsers
        "timezone", parse_timezone, show_timezone, reset_timezone
    },
    {
-       "cost_heap", parse_cost_heap,
-       show_cost_heap, reset_cost_heap
+       "cost_heap", parse_cost_heap, show_cost_heap, reset_cost_heap
    },
    {
-       "cost_index", parse_cost_index,
-       show_cost_index, reset_cost_index
+       "cost_index", parse_cost_index, show_cost_index, reset_cost_index
    },
    {
        "geqo", parse_geqo, show_geqo, reset_geqo