Changed the way symbols are defined in C in INFORMIX mode.

Added SQLSTATE macro closing bug #3961.
EXECUTE can return NOT FOUND so it should be checked here too.

diff --git a/src/interfaces/ecpg/ChangeLog b/src/interfaces/ecpg/ChangeLog
index 846c67cc0c9c4af19f65df459b4ac0ef9cc74b1b..7a2f8346a648928225de68884b6def934ea54643 100644 (file)
--- a/src/interfaces/ecpg/ChangeLog
+++ b/src/interfaces/ecpg/ChangeLog
@@ -2307,4 +2307,13 @@ Wed, 06 Feb 2008 09:04:48 +0100
        - Fixed segfault in ecpg when using an array element.
        - Free all memory in auto-prepare mode.
 
+Thu, 14 Feb 2008 13:11:34 +0100
+
+       - Added SQLSTATE macro closing bug #3961.
+       - EXECUTE can return NOT FOUND so it should be checked here too.
+       - Changed regression test accordingly.
+
+Fri, 15 Feb 2008 12:01:13 +0100
+
+       - Changed the way symbols are defined in C in INFORMIX mode.
 

diff --git a/src/interfaces/ecpg/include/datetime.h b/src/interfaces/ecpg/include/datetime.h
index 29cefba959257cd6c4ffcf8a8f3b470e67091ef3..03f2091db4af900eb34eb538b12b45fc7f96901c 100644 (file)
--- a/src/interfaces/ecpg/include/datetime.h
+++ b/src/interfaces/ecpg/include/datetime.h
@@ -5,7 +5,8 @@
 
 #include <ecpg_informix.h>
 
-typedef timestamp dtime_t;
-typedef interval intrvl_t;
+/* brought in by ecpg_informix.h nowadays
+ * typedef timestamp dtime_t;
+ * typedef interval intrvl_t; */
 
 #endif   /* ndef _ECPG_DATETIME_H */

diff --git a/src/interfaces/ecpg/include/decimal.h b/src/interfaces/ecpg/include/decimal.h
index 0f13dd1747531bf803911a7aed806f3e176d0de9..ae185a99c01113d351a4111dc92bd6ba9a5ab7f2 100644 (file)
--- a/src/interfaces/ecpg/include/decimal.h
+++ b/src/interfaces/ecpg/include/decimal.h
@@ -5,6 +5,7 @@
 
 #include <ecpg_informix.h>
 
-typedef decimal dec_t;
+/* brought in by ecpg_informix.h nowadays
+ * typedef decimal dec_t; */
 
 #endif   /* ndef _ECPG_DECIMAL_H */

diff --git a/src/interfaces/ecpg/include/ecpg_informix.h b/src/interfaces/ecpg/include/ecpg_informix.h
index c00914338dcd977fdf06e31fb03ada8a6af9fa1b..4153896c339e24be32429f8ac908373b77415455 100644 (file)
--- a/src/interfaces/ecpg/include/ecpg_informix.h
+++ b/src/interfaces/ecpg/include/ecpg_informix.h
@@ -82,6 +82,11 @@ extern int   dttofmtasc(timestamp *, char *, int, char *);
 extern int     intoasc(interval *, char *);
 extern int     dtcvfmtasc(char *, char *, timestamp *);
 
+/* we also define Informix datatypes here */
+typedef timestamp dtime_t;
+typedef interval intrvl_t;
+typedef decimal dec_t;
+
 #ifdef __cplusplus
 }
 #endif

diff --git a/src/interfaces/ecpg/include/ecpglib.h b/src/interfaces/ecpg/include/ecpglib.h
index 684348e71c080d6ced828d51622c5f9dab2d7655..d2706538bc335b2f6bed5c952d8edcb05dd0b0e1 100644 (file)
--- a/src/interfaces/ecpg/include/ecpglib.h
+++ b/src/interfaces/ecpg/include/ecpglib.h
@@ -58,7 +58,8 @@ void          sqlprint(void);
 
 /* define this for simplicity as well as compatibility */
 
-#define                  SQLCODE        sqlca.sqlcode
+#define                SQLCODE         sqlca.sqlcode
+#define                SQLSTATE        sqlca.sqlstate
 
 /* dynamic SQL */
 

diff --git a/src/interfaces/ecpg/preproc/pgc.l b/src/interfaces/ecpg/preproc/pgc.l
index bd0a7d265ed463f7622ffc08ea7f80e6cc3cda16..e7705dca6b53182788e188ebf09853a10c1bb55e 100644 (file)
--- a/src/interfaces/ecpg/preproc/pgc.l
+++ b/src/interfaces/ecpg/preproc/pgc.l
@@ -47,7 +47,6 @@ static void addlitchar (unsigned char);
 static void parse_include (void);
 static bool ecpg_isspace(char ch);
 static bool isdefine(void);
-static bool isinformixdefine(void);
 
 char *token_start;
 int state_before;
@@ -744,9 +743,11 @@ cppline                    {space}*#(.*\\{space})*.*{newline}
 <C>{identifier}        {
                                                const ScanKeyword               *keyword;
                                                        
-                                               /* Informix uses SQL defines only in SQL space */
-                                               /* however, some defines have to be taken care of for compatibility */
-                                               if ((!INFORMIX_MODE || !isinformixdefine()) && !isdefine())
+<<<<<<< pgc.l
+                                               if (!INFORMIX_MODE && !isdefine())
+=======
+                                               if (!isdefine())
+>>>>>>> 1.162
                                                {
                                                        keyword = ScanCKeywordLookup(yytext);
                                                        if (keyword != NULL)
@@ -1318,36 +1319,6 @@ static bool isdefine(void)
        return false;
 }
 
-static bool isinformixdefine(void)
-{
-       const char *new = NULL;
-
-       if (strcmp(yytext, "dec_t") == 0)
-               new = "decimal";
-       else if (strcmp(yytext, "intrvl_t") == 0)
-               new = "interval";
-       else if (strcmp(yytext, "dtime_t") == 0)
-                new = "timestamp";
-
-       if (new)
-       {
-               struct _yy_buffer *yb;
-
-               yb = mm_alloc(sizeof(struct _yy_buffer));
-
-               yb->buffer =  YY_CURRENT_BUFFER;
-               yb->lineno = yylineno;
-               yb->filename = mm_strdup(input_filename);
-               yb->next = yy_buffer;
-               yy_buffer = yb;
-
-               yy_scan_string(new);
-               return true;
-       }
-
-       return false;
-}
-
 /*
  * Called before any actual parsing is done
  */

diff --git a/src/interfaces/ecpg/preproc/preproc.y b/src/interfaces/ecpg/preproc/preproc.y
index cfc5a06eb4d65810a409465128208db5898c4db1..d9f5999c99ccb6267a89fd89cdb7cacf27be0eb1 100644 (file)
--- a/src/interfaces/ecpg/preproc/preproc.y
+++ b/src/interfaces/ecpg/preproc/preproc.y
@@ -807,7 +807,7 @@ stmt:  AlterDatabaseStmt            { output_statement($1, 0, ECPGst_normal); }
                | DropUserStmt          { output_statement($1, 0, ECPGst_normal); }
                | DropdbStmt            { output_statement($1, 0, ECPGst_normal); }
                | ExplainStmt           { output_statement($1, 0, ECPGst_normal); }
-               | ExecuteStmt           { output_statement($1, 0, ECPGst_execute); }
+               | ExecuteStmt           { output_statement($1, 1, ECPGst_execute); }
                | FetchStmt             { output_statement($1, 1, ECPGst_normal); }
                | GrantStmt             { output_statement($1, 0, ECPGst_normal); }
                | GrantRoleStmt         { output_statement($1, 0, ECPGst_normal); }