wine-commits October 2007

wine-commits@winehq.org

4 participants
996 discussions

Mikolaj Zalewski : kernel32/tests: Support for spaces in actctx test executable path.
by Alexandre Julliard 18 Oct '07

18 Oct '07

Module: wine Branch: master Commit: ba24af0c3445c117c011c3e0c500a7cc7d3e9c4e URL: http://source.winehq.org/git/wine.git/?a=commit;h=ba24af0c3445c117c011c3e0c… Author: Mikolaj Zalewski <mikolajz(a)google.com> Date: Wed Oct 17 11:38:55 2007 -0700 kernel32/tests: Support for spaces in actctx test executable path. --- dlls/kernel32/tests/actctx.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/kernel32/tests/actctx.c b/dlls/kernel32/tests/actctx.c index 0593909..d04e9ce 100644 --- a/dlls/kernel32/tests/actctx.c +++ b/dlls/kernel32/tests/actctx.c @@ -995,7 +995,7 @@ static void run_child_process(void) si.cb = sizeof(si); winetest_get_mainargs( &argv ); - sprintf(cmdline, "%s %s manifest1", argv[0], argv[1]); + sprintf(cmdline, "\"%s\" %s manifest1", argv[0], argv[1]); ok(CreateProcess(argv[0], cmdline, NULL, NULL, FALSE, 0, NULL, NULL, &si, &pi) != 0, "Could not create process: %u\n", GetLastError()); CloseHandle(pi.hThread);

1 0

Mikolaj Zalewski : kernel32/tests: Don' t close an invalid handle if the process is being debugged as this will raise an exception .
by Alexandre Julliard 18 Oct '07

18 Oct '07

Module: wine Branch: master Commit: 47e59d994fa7424fcce6d9cfbf59016f2f5c9680 URL: http://source.winehq.org/git/wine.git/?a=commit;h=47e59d994fa7424fcce6d9cfb… Author: Mikolaj Zalewski <mikolajz(a)google.com> Date: Wed Oct 17 11:08:51 2007 -0700 kernel32/tests: Don't close an invalid handle if the process is being debugged as this will raise an exception. --- dlls/kernel32/tests/actctx.c | 11 +++++++---- 1 files changed, 7 insertions(+), 4 deletions(-) diff --git a/dlls/kernel32/tests/actctx.c b/dlls/kernel32/tests/actctx.c index 5686089..0593909 100644 --- a/dlls/kernel32/tests/actctx.c +++ b/dlls/kernel32/tests/actctx.c @@ -16,7 +16,7 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ -#include <wine/test.h> +#include "wine/test.h" #include <winbase.h> #include <windef.h> #include <winnt.h> @@ -835,9 +835,12 @@ static void test_actctx(void) test_detailed_info(handle, &detailed_info1); test_info_in_assembly(handle, 1, &manifest1_info); - b = CloseHandle(handle); - ok(!b, "CloseHandle succeeded\n"); - ok(GetLastError() == ERROR_INVALID_HANDLE, "GetLastError() == %u\n", GetLastError()); + if (!IsDebuggerPresent()) /* CloseHandle will generate an exception if a debugger is present */ + { + b = CloseHandle(handle); + ok(!b, "CloseHandle succeeded\n"); + ok(GetLastError() == ERROR_INVALID_HANDLE, "GetLastError() == %u\n", GetLastError()); + } pReleaseActCtx(handle); }

1 0

James Hawkins : msi: Add handling for MSIMODIFY_INSERT for the streams table.
by Alexandre Julliard 18 Oct '07

18 Oct '07

Module: wine Branch: master Commit: 2694cf3ff8296ce16b572cbf9f2ff7e2783de13d URL: http://source.winehq.org/git/wine.git/?a=commit;h=2694cf3ff8296ce16b572cbf9… Author: James Hawkins <truiken(a)gmail.com> Date: Wed Oct 17 22:58:31 2007 -0500 msi: Add handling for MSIMODIFY_INSERT for the streams table. --- dlls/msi/streams.c | 32 ++++++++++++++++++++++++++++++-- 1 files changed, 30 insertions(+), 2 deletions(-) diff --git a/dlls/msi/streams.c b/dlls/msi/streams.c index 4a40a12..a4c0a52 100644 --- a/dlls/msi/streams.c +++ b/dlls/msi/streams.c @@ -280,8 +280,36 @@ static UINT STREAMS_get_column_info(struct tagMSIVIEW *view, static UINT STREAMS_modify(struct tagMSIVIEW *view, MSIMODIFY eModifyMode, MSIRECORD *rec, UINT row) { - FIXME("(%p, %d, %p): stub!\n", view, eModifyMode, rec); - return ERROR_SUCCESS; + UINT r; + + TRACE("%p %d %p\n", view, eModifyMode, rec); + + switch (eModifyMode) + { + case MSIMODIFY_INSERT: + r = STREAMS_insert_row(view, rec, FALSE); + break; + + case MSIMODIFY_VALIDATE_NEW: + case MSIMODIFY_INSERT_TEMPORARY: + case MSIMODIFY_UPDATE: + case MSIMODIFY_REFRESH: + case MSIMODIFY_ASSIGN: + case MSIMODIFY_REPLACE: + case MSIMODIFY_MERGE: + case MSIMODIFY_DELETE: + case MSIMODIFY_VALIDATE: + case MSIMODIFY_VALIDATE_FIELD: + case MSIMODIFY_VALIDATE_DELETE: + FIXME("%p %d %p - mode not implemented\n", view, eModifyMode, rec ); + r = ERROR_CALL_NOT_IMPLEMENTED; + break; + + default: + r = ERROR_INVALID_DATA; + } + + return r; } static UINT STREAMS_delete(struct tagMSIVIEW *view)

1 0

James Hawkins : msi: Save nonpersistent strings as holes in the string pool .
by Alexandre Julliard 18 Oct '07

18 Oct '07

Module: wine Branch: master Commit: 227438832349fc8be7ba1f10d4b1ec57e2e25891 URL: http://source.winehq.org/git/wine.git/?a=commit;h=227438832349fc8be7ba1f10d… Author: James Hawkins <truiken(a)gmail.com> Date: Wed Oct 17 22:57:18 2007 -0500 msi: Save nonpersistent strings as holes in the string pool. --- dlls/msi/string.c | 17 ++++++++++++----- dlls/msi/tests/db.c | 5 +---- 2 files changed, 13 insertions(+), 9 deletions(-) diff --git a/dlls/msi/string.c b/dlls/msi/string.c index 0cacf1e..fecd93f 100644 --- a/dlls/msi/string.c +++ b/dlls/msi/string.c @@ -457,20 +457,22 @@ UINT msi_strcmp( const string_table *st, UINT lval, UINT rval, UINT *res ) static void string_totalsize( const string_table *st, UINT *datasize, UINT *poolsize ) { - UINT i, len, max, holesize; + UINT i, len, holesize; if( st->strings[0].str || st->strings[0].persistent_refcount || st->strings[0].nonpersistent_refcount) ERR("oops. element 0 has a string\n"); *poolsize = 4; *datasize = 0; - max = 1; holesize = 0; for( i=1; i<st->maxcount; i++ ) { if( !st->strings[i].persistent_refcount ) - continue; - if( st->strings[i].str ) + { + TRACE("[%u] nonpersistent = %s\n", i, debugstr_w(st->strings[i].str)); + (*poolsize) += 4; + } + else if( st->strings[i].str ) { TRACE("[%u] = %s\n", i, debugstr_w(st->strings[i].str)); len = WideCharToMultiByte( st->codepage, 0, @@ -480,7 +482,6 @@ static void string_totalsize( const string_table *st, UINT *datasize, UINT *pool (*datasize) += len; if (len>0xffff) (*poolsize) += 4; - max = i + 1; (*poolsize) += holesize + 4; holesize = 0; } @@ -634,7 +635,13 @@ UINT msi_save_string_table( const string_table *st, IStorage *storage ) for( i=1; i<st->maxcount; i++ ) { if( !st->strings[i].persistent_refcount ) + { + pool[ n*2 ] = 0; + pool[ n*2 + 1] = 0; + n++; continue; + } + sz = datasize - used; r = msi_id2stringA( st, i, data+used, &sz ); if( r != ERROR_SUCCESS ) diff --git a/dlls/msi/tests/db.c b/dlls/msi/tests/db.c index 8def113..c620768 100644 --- a/dlls/msi/tests/db.c +++ b/dlls/msi/tests/db.c @@ -4253,10 +4253,7 @@ static void test_stringtable(void) sz = sizeof(buffer); r = MsiRecordGetString(hrec, 2, buffer, &sz); ok(r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r); - todo_wine - { - ok(!lstrcmp(buffer, "five"), "Expected five, got %s\n", buffer); - } + ok(!lstrcmp(buffer, "five"), "Expected five, got %s\n", buffer); r = MsiCloseHandle(hrec); ok(r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r);

1 0

James Hawkins : msi: Test adding nonpersistent strings to the string table.
by Alexandre Julliard 18 Oct '07

18 Oct '07

Module: wine Branch: master Commit: 36df6a0655ebf2209e13e7bcf1587b3291b95ca1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=36df6a0655ebf2209e13e7bcf… Author: James Hawkins <truiken(a)gmail.com> Date: Wed Oct 17 22:55:40 2007 -0500 msi: Test adding nonpersistent strings to the string table. --- dlls/msi/tests/db.c | 277 +++++++++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 277 insertions(+), 0 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=36df6a0655ebf2209e13e…

1 0

James Hawkins : msi: Return ERROR_FUNCTION_FAILED if a matching row is not found.
by Alexandre Julliard 18 Oct '07

18 Oct '07

Module: wine Branch: master Commit: 26f86c55c92105770bcb43601e50ec3f54f8bee6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=26f86c55c92105770bcb43601… Author: James Hawkins <truiken(a)gmail.com> Date: Wed Oct 17 22:54:39 2007 -0500 msi: Return ERROR_FUNCTION_FAILED if a matching row is not found. --- dlls/msi/table.c | 2 +- dlls/msi/tests/db.c | 10 ++-------- 2 files changed, 3 insertions(+), 9 deletions(-) diff --git a/dlls/msi/table.c b/dlls/msi/table.c index 692f4a5..a817098 100644 --- a/dlls/msi/table.c +++ b/dlls/msi/table.c @@ -1560,7 +1560,7 @@ static UINT msi_table_update(struct tagMSIVIEW *view, MSIRECORD *rec, UINT row) if (r != ERROR_SUCCESS) { ERR("can't find row to modify\n"); - return ERROR_SUCCESS; + return ERROR_FUNCTION_FAILED; } /* the row cannot be changed */ diff --git a/dlls/msi/tests/db.c b/dlls/msi/tests/db.c index 8887954..76d1617 100644 --- a/dlls/msi/tests/db.c +++ b/dlls/msi/tests/db.c @@ -841,10 +841,7 @@ static void test_viewmodify(void) ok(r == ERROR_SUCCESS, "MsiRecordSetInteger failed\n"); r = MsiViewModify(hview, MSIMODIFY_UPDATE, hrec); - todo_wine - { - ok(r == ERROR_FUNCTION_FAILED, "MsiViewModify failed\n"); - } + ok(r == ERROR_FUNCTION_FAILED, "MsiViewModify failed\n"); r = MsiCloseHandle(hrec); ok(r == ERROR_SUCCESS, "failed to close record\n"); @@ -892,10 +889,7 @@ static void test_viewmodify(void) ok(r == ERROR_SUCCESS, "failed to set string\n"); r = MsiViewModify(hview, MSIMODIFY_UPDATE, hrec); - todo_wine - { - ok(r == ERROR_FUNCTION_FAILED, "Expected ERROR_FUNCTION_FAILED, got %d\n", r); - } + ok(r == ERROR_FUNCTION_FAILED, "Expected ERROR_FUNCTION_FAILED, got %d\n", r); r = MsiCloseHandle(hrec); ok(r == ERROR_SUCCESS, "failed to close record\n");

1 0

James Hawkins : msi: Return ERROR_FUNCTION_FAILED when inserting duplicate keys.
by Alexandre Julliard 18 Oct '07

18 Oct '07

Module: wine Branch: master Commit: 46ec7ee183129cc1099a001febc5e77225ecacfb URL: http://source.winehq.org/git/wine.git/?a=commit;h=46ec7ee183129cc1099a001fe… Author: James Hawkins <truiken(a)gmail.com> Date: Wed Oct 17 22:53:54 2007 -0500 msi: Return ERROR_FUNCTION_FAILED when inserting duplicate keys. --- dlls/msi/table.c | 2 +- dlls/msi/tests/db.c | 2 -- 2 files changed, 1 insertions(+), 3 deletions(-) diff --git a/dlls/msi/table.c b/dlls/msi/table.c index c5850e3..692f4a5 100644 --- a/dlls/msi/table.c +++ b/dlls/msi/table.c @@ -1490,7 +1490,7 @@ static UINT table_validate_new( MSITABLEVIEW *tv, MSIRECORD *rec ) /* check there's no duplicate keys */ r = msi_table_find_row( tv, rec, &row ); if (r == ERROR_SUCCESS) - return ERROR_INVALID_DATA; + return ERROR_FUNCTION_FAILED; return ERROR_SUCCESS; } diff --git a/dlls/msi/tests/db.c b/dlls/msi/tests/db.c index 293cf0c..8887954 100644 --- a/dlls/msi/tests/db.c +++ b/dlls/msi/tests/db.c @@ -791,10 +791,8 @@ static void test_viewmodify(void) ok(r == ERROR_SUCCESS, "MsiViewExecute failed\n"); /* should fail ... */ - todo_wine { r = MsiViewModify(hview, MSIMODIFY_INSERT_TEMPORARY, hrec ); ok(r == ERROR_FUNCTION_FAILED, "MsiViewModify failed\n"); - } r = MsiCloseHandle(hrec); ok(r == ERROR_SUCCESS, "failed to close record\n");

1 0

Dan Hipschman : widl: Detect conformant arrays of user types correctly.
by Alexandre Julliard 18 Oct '07

18 Oct '07

Module: wine Branch: master Commit: 1d0f9378bc855652bf45db0e29b07fa85724dbd1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1d0f9378bc855652bf45db0e2… Author: Dan Hipschman <dsh(a)linux.ucla.edu> Date: Wed Oct 17 18:06:26 2007 -0700 widl: Detect conformant arrays of user types correctly. --- dlls/rpcrt4/tests/server.c | 54 ++++++++++++++++++++++++++++++++++++++++++ dlls/rpcrt4/tests/server.idl | 16 ++++++++++++ tools/widl/header.c | 4 ++- tools/widl/parser.y | 41 ++++++++++++++------------------ tools/widl/typegen.c | 16 ++++++++++++ 5 files changed, 107 insertions(+), 24 deletions(-) diff --git a/dlls/rpcrt4/tests/server.c b/dlls/rpcrt4/tests/server.c index d8ecff0..97f2e23 100644 --- a/dlls/rpcrt4/tests/server.c +++ b/dlls/rpcrt4/tests/server.c @@ -48,6 +48,14 @@ midl_user_free(void __RPC_FAR *p) free(p); } +static char * +xstrdup(const char *s) +{ + char *d = HeapAlloc(GetProcessHeap(), 0, strlen(s) + 1); + strcpy(d, s); + return d; +} + int s_int_return(void) { @@ -264,6 +272,26 @@ s_square_puint(puint_t p) } int +s_sum_puints(puints_t *p) +{ + int sum = 0; + int i; + for (i = 0; i < p->n; ++i) + sum += atoi(p->ps[i]); + return sum; +} + +int +s_sum_cpuints(cpuints_t *p) +{ + int sum = 0; + int i; + for (i = 0; i < p->n; ++i) + sum += atoi(p->ps[i]); + return sum; +} + +int s_dot_copy_vectors(vector_t u, vector_t v) { return u.x * v.x + u.y * v.y + u.z * v.z; @@ -724,9 +752,35 @@ pointer_tests(void) test_list_t *list = make_list(make_list(make_list(null_list()))); test_us_t tus = {{p1}}; int *pa[4]; + puints_t pus; + cpuints_t cpus; ok(test_list_length(list) == 3, "RPC test_list_length\n"); ok(square_puint(p1) == 121, "RPC square_puint\n"); + pus.n = 4; + pus.ps = HeapAlloc(GetProcessHeap(), 0, pus.n * sizeof pus.ps[0]); + pus.ps[0] = xstrdup("5"); + pus.ps[1] = xstrdup("6"); + pus.ps[2] = xstrdup("7"); + pus.ps[3] = xstrdup("8"); + ok(sum_puints(&pus) == 26, "RPC sum_puints\n"); + HeapFree(GetProcessHeap(), 0, pus.ps[0]); + HeapFree(GetProcessHeap(), 0, pus.ps[1]); + HeapFree(GetProcessHeap(), 0, pus.ps[2]); + HeapFree(GetProcessHeap(), 0, pus.ps[3]); + HeapFree(GetProcessHeap(), 0, pus.ps); + cpus.n = 4; + cpus.ps = HeapAlloc(GetProcessHeap(), 0, cpus.n * sizeof cpus.ps[0]); + cpus.ps[0] = xstrdup("5"); + cpus.ps[1] = xstrdup("6"); + cpus.ps[2] = xstrdup("7"); + cpus.ps[3] = xstrdup("8"); + ok(sum_cpuints(&cpus) == 26, "RPC sum_puints\n"); + HeapFree(GetProcessHeap(), 0, cpus.ps[0]); + HeapFree(GetProcessHeap(), 0, cpus.ps[1]); + HeapFree(GetProcessHeap(), 0, cpus.ps[2]); + HeapFree(GetProcessHeap(), 0, cpus.ps[3]); + HeapFree(GetProcessHeap(), 0, cpus.ps); ok(square_test_us(&tus) == 121, "RPC square_test_us\n"); pa[0] = &a[0]; diff --git a/dlls/rpcrt4/tests/server.idl b/dlls/rpcrt4/tests/server.idl index 6233a1f..807bf44 100644 --- a/dlls/rpcrt4/tests/server.idl +++ b/dlls/rpcrt4/tests/server.idl @@ -153,6 +153,22 @@ cpp_quote("#endif") typedef [wire_marshal(int)] void *puint_t; int square_puint(puint_t p); + + typedef struct + { + [size_is(n)] puint_t *ps; + int n; + } puints_t; + + /* Same thing as puints_t, but make it complex (needs padding). */ + typedef struct + { + [size_is(n)] puint_t *ps; + char n; + } cpuints_t; + + int sum_puints(puints_t *p); + int sum_cpuints(cpuints_t *p); int dot_copy_vectors(vector_t u, vector_t v); typedef struct wire_us *wire_us_t; diff --git a/tools/widl/header.c b/tools/widl/header.c index 2643f49..8de9b9f 100644 --- a/tools/widl/header.c +++ b/tools/widl/header.c @@ -101,7 +101,9 @@ int is_void(const type_t *t) int is_conformant_array(const type_t *t) { - return t->type == RPC_FC_CARRAY || t->type == RPC_FC_CVARRAY; + return t->type == RPC_FC_CARRAY + || t->type == RPC_FC_CVARRAY + || (t->type == RPC_FC_BOGUS_ARRAY && t->size_is); } void write_guid(FILE *f, const char *guid_prefix, const char *name, const UUID *uuid) diff --git a/tools/widl/parser.y b/tools/widl/parser.y index 96e9891..6a93995 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -1434,24 +1434,24 @@ static void set_type(var_t *v, type_t *type, int ptr_level, array_dims_t *arr, if (is_array(v->type)) { const type_t *rt = v->type->ref; - switch (rt->type) - { - case RPC_FC_BOGUS_STRUCT: - case RPC_FC_NON_ENCAPSULATED_UNION: - case RPC_FC_ENCAPSULATED_UNION: - case RPC_FC_ENUM16: + if (is_user_type(rt)) v->type->type = RPC_FC_BOGUS_ARRAY; - break; - /* FC_RP should be above, but widl overuses these, and will break things. */ - case RPC_FC_UP: - case RPC_FC_RP: - if (rt->ref->type == RPC_FC_IP) - v->type->type = RPC_FC_BOGUS_ARRAY; - break; - default: - if (is_user_type(rt)) - v->type->type = RPC_FC_BOGUS_ARRAY; - } + else + switch (rt->type) + { + case RPC_FC_BOGUS_STRUCT: + case RPC_FC_NON_ENCAPSULATED_UNION: + case RPC_FC_ENCAPSULATED_UNION: + case RPC_FC_ENUM16: + v->type->type = RPC_FC_BOGUS_ARRAY; + break; + /* FC_RP should be above, but widl overuses these, and will break things. */ + case RPC_FC_UP: + case RPC_FC_RP: + if (rt->ref->type == RPC_FC_IP) + v->type->type = RPC_FC_BOGUS_ARRAY; + break; + } } } @@ -1859,6 +1859,7 @@ static int get_struct_type(var_list_t *fields) case RPC_FC_OP: case RPC_FC_CARRAY: case RPC_FC_CVARRAY: + case RPC_FC_BOGUS_ARRAY: has_pointer = 1; break; @@ -1897,15 +1898,9 @@ static int get_struct_type(var_list_t *fields) /* fallthru - treat it as complex */ /* as soon as we see one of these these members, it's bogus... */ - case RPC_FC_IP: case RPC_FC_ENCAPSULATED_UNION: case RPC_FC_NON_ENCAPSULATED_UNION: - case RPC_FC_TRANSMIT_AS: - case RPC_FC_REPRESENT_AS: - case RPC_FC_PAD: - case RPC_FC_EMBEDDED_COMPLEX: case RPC_FC_BOGUS_STRUCT: - case RPC_FC_BOGUS_ARRAY: return RPC_FC_BOGUS_STRUCT; } } diff --git a/tools/widl/typegen.c b/tools/widl/typegen.c index 53ec78c..2245524 100644 --- a/tools/widl/typegen.c +++ b/tools/widl/typegen.c @@ -1744,6 +1744,22 @@ static size_t write_struct_tfs(FILE *file, type_t *type, type_t *ft = f->type; if (is_ptr(ft)) write_pointer_tfs(file, ft, tfsoff); + else if (!ft->declarray && is_conformant_array(ft)) + { + unsigned int absoff = ft->typestring_offset; + short reloff = absoff - (*tfsoff + 2); + int ptr_type = get_attrv(f->attrs, ATTR_POINTERTYPE); + /* FIXME: We need to store pointer attributes for arrays + so we don't lose pointer_default info. */ + if (ptr_type == 0) + ptr_type = RPC_FC_UP; + print_file(file, 0, "/* %d */\n", *tfsoff); + print_file(file, 2, "0x%x, 0x0,\t/* %s */\n", ptr_type, + string_of_type(ptr_type)); + print_file(file, 2, "NdrFcShort(0x%hx),\t/* Offset= %hd (%u) */\n", + reloff, reloff, absoff); + *tfsoff += 4; + } } if (type->ptrdesc == *tfsoff) type->ptrdesc = 0;

1 0

Dan Hipschman : widl: Remove printf format strings that aren' t really format strings.
by Alexandre Julliard 18 Oct '07

18 Oct '07

Module: wine Branch: master Commit: 4fb7a0e40ed9c8ef47faa1d4ce4f26a545614dcb URL: http://source.winehq.org/git/wine.git/?a=commit;h=4fb7a0e40ed9c8ef47faa1d4c… Author: Dan Hipschman <dsh(a)linux.ucla.edu> Date: Wed Oct 17 18:05:37 2007 -0700 widl: Remove printf format strings that aren't really format strings. --- tools/widl/typegen.c | 4 ++-- tools/widl/widl.c | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/tools/widl/typegen.c b/tools/widl/typegen.c index bf24d3f..53ec78c 100644 --- a/tools/widl/typegen.c +++ b/tools/widl/typegen.c @@ -2501,7 +2501,7 @@ void print_phase_basetype(FILE *file, int indent, enum remoting_phase phase, fprintf(file, " *)_StubMsg.Buffer = *"); else fprintf(file, " *)_StubMsg.Buffer = "); - fprintf(file, varname); + fprintf(file, "%s", varname); fprintf(file, ";\n"); } else if (phase == PHASE_UNMARSHAL) @@ -2510,7 +2510,7 @@ void print_phase_basetype(FILE *file, int indent, enum remoting_phase phase, print_file(file, indent, ""); else print_file(file, indent, "*"); - fprintf(file, varname); + fprintf(file, "%s", varname); if (pass == PASS_IN && is_ptr(type)) fprintf(file, " = ("); else diff --git a/tools/widl/widl.c b/tools/widl/widl.c index 53c9e29..37f1a5e 100644 --- a/tools/widl/widl.c +++ b/tools/widl/widl.c @@ -422,13 +422,13 @@ int main(int argc,char *argv[]) idfile_name = xstrdup(optarg); break; case 'V': - printf(version_string); + printf("%s", version_string); return 0; case 'W': pedantic = 1; break; default: - fprintf(stderr, usage); + fprintf(stderr, "%s", usage); return 1; } } @@ -458,7 +458,7 @@ int main(int argc,char *argv[]) input_name = xstrdup(argv[optind]); } else { - fprintf(stderr, usage); + fprintf(stderr, "%s", usage); return 1; }

1 0

Dan Hipschman : widl: Generate dlldata files.
by Alexandre Julliard 18 Oct '07

18 Oct '07

Module: wine Branch: master Commit: 40e90aae25198e7a059520c0258eefe3e817fe00 URL: http://source.winehq.org/git/wine.git/?a=commit;h=40e90aae25198e7a059520c02… Author: Dan Hipschman <dsh(a)linux.ucla.edu> Date: Wed Oct 17 18:03:57 2007 -0700 widl: Generate dlldata files. --- tools/widl/parser.y | 1 + tools/widl/utils.c | 29 ++++++++ tools/widl/utils.h | 1 + tools/widl/widl.c | 181 +++++++++++++++++++++++++++++++++++++++++++++++---- tools/widl/widl.h | 3 + 5 files changed, 202 insertions(+), 13 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=40e90aae25198e7a05952…

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits October 2007