]> granicus.if.org Git - postgresql/commitdiff
Do not try to find local timezone in DecodeDateTime() because the
authorMichael Meskes <meskes@postgresql.org>
Mon, 21 May 2007 07:04:00 +0000 (07:04 +0000)
committerMichael Meskes <meskes@postgresql.org>
Mon, 21 May 2007 07:04:00 +0000 (07:04 +0000)
result is not used anyway. This also fixes Vista's build problems.

src/interfaces/ecpg/ChangeLog
src/interfaces/ecpg/pgtypeslib/dt_common.c

index 32ba47d760d6af85a807ca6705f8982d3def8b40..b866975d3fdda4343f2fb7cd0378c9d90f24fcca 100644 (file)
@@ -2194,8 +2194,8 @@ Thu, 10 May 09:42:42 CEST 2007
        - Synced parser and keyword list.
        - Renamed update test so it hopefully runs on Vista.
        
-Th, 17 May 2007 22:03:40 +0200
+Sun, 20 May 2007 22:01:11 +0200
 
-       - Removed mktime() call to hopefully get rid of some compatibility
-         problems.
+       - Do not try to find local timezone in DecodeDateTime() because the
+         result is not used anyway. This also fixes Vista's build problems.
        - Set ecpg version to 4.3.1.
index 8bdc64b000176a03414a3bbfa32254bb5e8075c9..f4df2fb90c6a1926dfb3f0517f7225b8185ea5ed 100644 (file)
@@ -1,4 +1,4 @@
-/* $PostgreSQL: pgsql/src/interfaces/ecpg/pgtypeslib/dt_common.c,v 1.38 2007/05/20 11:30:30 meskes Exp $ */
+/* $PostgreSQL: pgsql/src/interfaces/ecpg/pgtypeslib/dt_common.c,v 1.39 2007/05/21 07:04:00 meskes Exp $ */
 
 #include "postgres_fe.h"
 
@@ -2528,20 +2528,12 @@ DecodeDateTime(char **field, int *ftype, int nf,
                if (tm->tm_mday < 1 || tm->tm_mday > day_tab[isleap(tm->tm_year)][tm->tm_mon - 1])
                        return -1;
 
-               /* timezone not specified? then find local timezone if possible */
-               if ((fmask & DTK_DATE_M) == DTK_DATE_M && tzp != NULL && !(fmask & DTK_M(TZ)))
-               {
-                       /*
-                        * daylight savings time modifier but no standard timezone? then
-                        * error
-                        */
-                       if (fmask & DTK_M(DTZMOD))
+               /* backend tried to find local timezone here 
+                * but we don't use the result afterwards anyway
+                * so we only check for this error:
+                * daylight savings time modifier but no standard timezone? */
+               if ((fmask & DTK_DATE_M) == DTK_DATE_M && tzp != NULL && !(fmask & DTK_M(TZ)) && (fmask & DTK_M(DTZMOD)))
                                return -1;
-
-                       /* test to see if this is the reason for the error on Vista
-                        * doesn't seem to be used anyway
-                        * *tzp = DetermineLocalTimeZone(tm); */
-               }
        }
 
        return 0;