Remove formatter's assumption that year would never exceed four digits.

Enforce MAXTZLEN for all datestyles, not just some.  Remove macro
definitions that were redundant with datetime.h.

diff --git a/src/backend/utils/adt/datetime.c b/src/backend/utils/adt/datetime.c
index 81a3d96b8aa9daa6bcb90d1262ab9264448f9daf..7095f24de7313023f9210f7b5dc9cc3b36a58331 100644 (file)
--- a/src/backend/utils/adt/datetime.c
+++ b/src/backend/utils/adt/datetime.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *       $Header: /cvsroot/pgsql/src/backend/utils/adt/datetime.c,v 1.65 2001/06/18 16:14:43 momjian Exp $
+ *       $Header: /cvsroot/pgsql/src/backend/utils/adt/datetime.c,v 1.66 2001/07/10 01:41:47 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -25,6 +25,9 @@
 #include "utils/guc.h"
 #include "utils/datetime.h"
 
+
+#define ROUND_ALL 1
+
 static int DecodeNumber(int flen, char *field,
                         int fmask, int *tmask,
                         struct tm * tm, double *fsec, int *is2digits);
@@ -36,15 +39,12 @@ static int DecodeTime(char *str, int fmask, int *tmask,
 static int     DecodeTimezone(char *str, int *tzp);
 static datetkn *datebsearch(char *key, datetkn *base, unsigned int nel);
 static int     DecodeDate(char *str, int fmask, int *tmask, struct tm * tm);
-
-#define ROUND_ALL 0
-
 static int     DecodePosixTimezone(char *str, int *val);
 
+
 int                    day_tab[2][13] = {
        {31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31, 0},
-{31, 29, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31, 0}};
-
+       {31, 29, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31, 0}};
 
 char      *months[] = {"Jan", "Feb", "Mar", "Apr", "May", "Jun",
 "Jul", "Aug", "Sep", "Oct", "Nov", "Dec", NULL};
@@ -53,28 +53,13 @@ char           *days[] = {"Sunday", "Monday", "Tuesday", "Wednesday",
 "Thursday", "Friday", "Saturday", NULL};
 
 
-#define UTIME_MINYEAR (1901)
-#define UTIME_MINMONTH (12)
-#define UTIME_MINDAY (14)
-#define UTIME_MAXYEAR (2038)
-#define UTIME_MAXMONTH (01)
-#define UTIME_MAXDAY (18)
-
-#define IS_VALID_UTIME(y,m,d) (((y > UTIME_MINYEAR) \
- || ((y == UTIME_MINYEAR) && ((m > UTIME_MINMONTH) \
-  || ((m == UTIME_MINMONTH) && (d >= UTIME_MINDAY))))) \
- && ((y < UTIME_MAXYEAR) \
- || ((y == UTIME_MAXYEAR) && ((m < UTIME_MAXMONTH) \
-  || ((m == UTIME_MAXMONTH) && (d <= UTIME_MAXDAY))))))
-
-
 /*****************************************************************************
  *      PRIVATE ROUTINES                                                                                                                *
  *****************************************************************************/
 
 /* definitions for squeezing values into "value" */
-#define ABS_SIGNBIT            (char) 0200
-#define VALMASK                        (char) 0177
+#define ABS_SIGNBIT            ((char) 0200)
+#define VALMASK                        ((char) 0177)
 #define NEG(n)                 ((n)|ABS_SIGNBIT)
 #define SIGNEDCHAR(c)  ((c)&ABS_SIGNBIT? -((c)&VALMASK): (c))
 #define FROMVAL(tp)            (-SIGNEDCHAR((tp)->value) * 10) /* uncompress */
@@ -2112,7 +2097,7 @@ EncodeDateTime(struct tm * tm, double fsec, int *tzp, char **tzn, int style, cha
                        {
                                sprintf(str, "%04d-%02d-%02d %02d:%02d:",
                                                tm->tm_year, tm->tm_mon, tm->tm_mday, tm->tm_hour, tm->tm_min);
-                               sprintf((str + 17), ((fsec != 0) ? "%05.2f" : "%02.0f"), sec);
+                               sprintf((str + strlen(str)), ((fsec != 0) ? "%05.2f" : "%02.0f"), sec);
 
                                if ((*tzn != NULL) && (tm->tm_isdst >= 0))
                                {
@@ -2154,11 +2139,7 @@ EncodeDateTime(struct tm * tm, double fsec, int *tzp, char **tzn, int style, cha
                                                tm->tm_year, tm->tm_hour, tm->tm_min, sec);
 
                                if ((*tzn != NULL) && (tm->tm_isdst >= 0))
-                               {
-                                       strcpy((str + 22), " ");
-                                       strcpy((str + 23), *tzn);
-                               }
-
+                                       sprintf((str + strlen(str)), " %.*s", MAXTZLEN, *tzn);
                        }
                        else
                                sprintf((str + 5), "/%04d %02d:%02d %s",
@@ -2174,11 +2155,7 @@ EncodeDateTime(struct tm * tm, double fsec, int *tzp, char **tzn, int style, cha
                                                tm->tm_year, tm->tm_hour, tm->tm_min, sec);
 
                                if ((*tzn != NULL) && (tm->tm_isdst >= 0))
-                               {
-                                       strcpy((str + 22), " ");
-                                       strcpy((str + 23), *tzn);
-                               }
-
+                                       sprintf((str + strlen(str)), " %.*s", MAXTZLEN, *tzn);
                        }
                        else
                                sprintf((str + 5), ".%04d %02d:%02d %s",
@@ -2206,21 +2183,14 @@ EncodeDateTime(struct tm * tm, double fsec, int *tzp, char **tzn, int style, cha
                                {
                                        sprintf((str + 16), ":%05.2f %04d", sec, tm->tm_year);
                                        if ((*tzn != NULL) && (tm->tm_isdst >= 0))
-                                       {
-                                               strcpy((str + 27), " ");
-                                               StrNCpy((str + 28), *tzn, MAXTZLEN + 1);
-                                       }
+                                               sprintf((str + strlen(str)), " %.*s", MAXTZLEN, *tzn);
                                }
                                else
                                {
                                        sprintf((str + 16), ":%02.0f %04d", sec, tm->tm_year);
                                        if ((*tzn != NULL) && (tm->tm_isdst >= 0))
-                                       {
-                                               strcpy((str + 24), " ");
-                                               StrNCpy((str + 25), *tzn, MAXTZLEN + 1);
-                                       }
+                                               sprintf((str + strlen(str)), " %.*s", MAXTZLEN, *tzn);
                                }
-
                        }
                        else
                        {