MSI: remove more types from the parser

Mike McCormack mike at codeweavers.com
Fri May 27 01:45:26 CDT 2005


ChangeLog:
* remove more types from the parser
-------------- next part --------------
diff -urp dlls/msi.old/insert.c dlls/msi/insert.c
--- dlls/msi.old/insert.c	2005-05-27 15:36:54.000000000 +0900
+++ dlls/msi/insert.c	2005-05-27 15:40:29.000000000 +0900
@@ -44,7 +44,7 @@ typedef struct tagMSIINSERTVIEW
     MSIDATABASE     *db;
     BOOL             bIsTemp;
     MSIVIEW         *sv;
-    value_list      *vals;
+    column_info     *vals;
 } MSIINSERTVIEW;
 
 static UINT INSERT_fetch_int( struct tagMSIVIEW *view, UINT row, UINT col, UINT *val )
@@ -62,7 +62,7 @@ static UINT INSERT_fetch_int( struct tag
  * Merge a value_list and a record to create a second record.
  * Replace wildcard entries in the valuelist with values from the record
  */
-static MSIRECORD *INSERT_merge_record( UINT fields, value_list *vl, MSIRECORD *rec )
+static MSIRECORD *INSERT_merge_record( UINT fields, column_info *vl, MSIRECORD *rec )
 {
     MSIRECORD *merged;
     DWORD wildcard_count = 1, i;
@@ -255,7 +255,7 @@ MSIVIEWOPS insert_ops =
 };
 
 UINT INSERT_CreateView( MSIDATABASE *db, MSIVIEW **view, LPWSTR table,
-                        column_info *columns, value_list *values, BOOL temp )
+                        column_info *columns, column_info *values, BOOL temp )
 {
     MSIINSERTVIEW *iv = NULL;
     UINT r;
diff -urp dlls/msi.old/query.h dlls/msi/query.h
--- dlls/msi.old/query.h	2005-05-27 15:36:54.000000000 +0900
+++ dlls/msi/query.h	2005-05-27 15:40:29.000000000 +0900
@@ -68,12 +68,6 @@ typedef struct _column_info
     struct _column_info *next;
 } column_info;
 
-typedef struct _string_list
-{
-    LPWSTR string;
-    struct _string_list *next;
-} string_list;
-
 struct complex_expr
 {
     UINT op;
@@ -95,19 +89,6 @@ struct expr
     } u;
 };
 
-typedef struct _value_list
-{
-    struct expr *val;
-    struct _value_list *next;
-} value_list;
-
-typedef struct _column_assignment
-{
-    column_info *col_list;
-    value_list *val_list;
-} column_assignment;
-
-
 UINT MSI_ParseSQL( MSIDATABASE *db, LPCWSTR command, MSIVIEW **phview,
                    struct list *mem );
 
@@ -128,7 +109,7 @@ UINT CREATE_CreateView( MSIDATABASE *db,
                         column_info *col_info, BOOL temp );
 
 UINT INSERT_CreateView( MSIDATABASE *db, MSIVIEW **view, LPWSTR table,
-                        column_info *columns, value_list *values, BOOL temp );
+                        column_info *columns, column_info *values, BOOL temp );
 
 UINT UPDATE_CreateView( MSIDATABASE *db, MSIVIEW **, LPWSTR table,
                         column_info *list, struct expr *expr );
diff -urp dlls/msi.old/sql.y dlls/msi/sql.y
--- dlls/msi.old/sql.y	2005-05-27 15:36:54.000000000 +0900
+++ dlls/msi/sql.y	2005-05-27 15:42:06.000000000 +0900
@@ -32,7 +32,6 @@
 #include "query.h"
 #include "wine/list.h"
 #include "wine/debug.h"
-#include "wine/unicode.h"
 
 #define YYLEX_PARAM info
 #define YYPARSE_PARAM info
@@ -74,7 +73,6 @@ static struct expr * EXPR_wildcard( void
     struct sql_str str;
     LPWSTR string;
     column_info *column_list;
-    value_list *val_list;
     MSIVIEW *query;
     struct expr *expr;
     USHORT column_type;
@@ -126,12 +124,11 @@ static struct expr * EXPR_wildcard( void
 
 %type <string> table id
 %type <column_list> selcollist column column_and_type column_def table_def
-%type <column_list> column_assignment update_assign_list
+%type <column_list> column_assignment update_assign_list constlist
 %type <query> query from fromtable selectfrom unorderedsel
 %type <query> oneupdate onedelete oneselect onequery onecreate oneinsert
 %type <expr> expr val column_val const_val
 %type <column_type> column_type data_type data_type_l data_count
-%type <val_list> constlist
 
 %%
 
@@ -498,25 +495,18 @@ val:
 constlist:
     const_val
         {
-            value_list *vals;
-
-            vals = parser_alloc( info, sizeof *vals );
-            if( !vals )
+            $$ = parser_alloc_column( info, NULL, NULL );
+            if( !$$ )
                 YYABORT;
-            vals->val = $1;
-            vals->next = NULL;
-            $$ = vals;
+            $$->val = $1;
         }
   | const_val TK_COMMA constlist
         {
-            value_list *vals;
-
-            vals = parser_alloc( info, sizeof *vals );
-            if( !vals )
+            $$ = parser_alloc_column( info, NULL, NULL );
+            if( !$$ )
                 YYABORT;
-            vals->val = $1;
-            vals->next = $3;
-            $$ = vals;
+            $$->val = $1;
+            $$->next = $3;
         }
     ;
 


More information about the wine-patches mailing list