August 2019 - Wine-commits - list.winehq.org

Jacek Caban : jscript: Support properties argument in Object.create.
by Alexandre Julliard 21 Aug '19

21 Aug '19

Module: wine Branch: master Commit: e458e706360f8556367202174953a716cab5e0e9 URL: https://source.winehq.org/git/wine.git/?a=commit;h=e458e706360f855636720217… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Aug 21 12:31:33 2019 +0200 jscript: Support properties argument in Object.create. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/jscript/object.c | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/dlls/jscript/object.c b/dlls/jscript/object.c index 7794f1d..123541e 100644 --- a/dlls/jscript/object.c +++ b/dlls/jscript/object.c @@ -598,11 +598,6 @@ static HRESULT Object_create(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, TRACE("(%s)\n", debugstr_jsval(argv[0])); - if(argc > 1) { - FIXME("Unsupported properties argument %s\n", debugstr_jsval(argv[1])); - return E_NOTIMPL; - } - if(argc && is_object_instance(argv[0])) { if(get_object(argv[0])) proto = to_jsdisp(get_object(argv[0])); @@ -615,13 +610,18 @@ static HRESULT Object_create(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, return E_INVALIDARG; } - if(r) { - hres = create_dispex(ctx, NULL, proto, &obj); - if(FAILED(hres)) - return hres; + hres = create_dispex(ctx, NULL, proto, &obj); + if(FAILED(hres)) + return hres; + + if(argc >= 2 && !is_undefined(argv[1])) + hres = jsdisp_define_properties(ctx, obj, argv[1]); + + if(SUCCEEDED(hres) && r) *r = jsval_obj(obj); - } - return S_OK; + else + jsdisp_release(obj); + return hres; } static HRESULT Object_getPrototypeOf(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags,

1 0

Jacek Caban : mshtml/tests: Add Object.defineProperties tests.
by Alexandre Julliard 21 Aug '19

21 Aug '19

Module: wine Branch: master Commit: 98cf6a6ac79dd0da64450b48c638eba0c7f4d57c URL: https://source.winehq.org/git/wine.git/?a=commit;h=98cf6a6ac79dd0da64450b48… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Aug 21 12:31:44 2019 +0200 mshtml/tests: Add Object.defineProperties tests. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/mshtml/tests/es5.js | 43 +++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 43 insertions(+) diff --git a/dlls/mshtml/tests/es5.js b/dlls/mshtml/tests/es5.js index 85a6899..11b68e5 100644 --- a/dlls/mshtml/tests/es5.js +++ b/dlls/mshtml/tests/es5.js @@ -507,6 +507,48 @@ function test_defineProperty() { next_test(); } +function test_defineProperties() { + var o, defined, descs; + + descs = { + acc_prop: { get: function() { return 1; } }, + prop: { value: 2 }, + e: { enumerable: true } + }; + + o = new Object(); + defined = Object.defineProperties(o, descs); + ok(defined === o, "defined != o"); + ok(o.acc_prop === 1, "o.acc_prop = " + o.acc_prop); + ok(o.prop === 2, "o.prop = " + o.prop); + ok(o.e === undefined, "o.e = " + o.e); + ok("e" in o, "e is not in o"); + test_own_data_prop_desc(o, "prop", false, false, false); + test_own_data_prop_desc(o, "e", false, true, false); + for(var p in o) ok(p === "e", "p = " + p); + + o = new Object(); + Object.defineProperties(o, 1); + for(var p in o) ok(false, "o has property " + p); + + o = Object.create(null, descs); + ok(o.acc_prop === 1, "o.acc_prop = " + o.acc_prop); + ok(o.prop === 2, "o.prop = " + o.prop); + ok(o.e === undefined, "o.e = " + o.e); + ok("e" in o, "e is not in o"); + test_own_data_prop_desc(o, "prop", false, false, false); + test_own_data_prop_desc(o, "e", false, true, false); + for(var p in o) ok(p === "e", "p = " + p); + + var desc_proto = new Object(); + desc_proto.proto_prop = { value: true, enumerable: true }; + descs = Object.create(desc_proto); + o = Object.create(null, descs); + ok(!("proto_prop" in o), "proto_prop is in o"); + + next_test(); +} + function test_property_definitions() { var obj, val, i, arr; @@ -811,6 +853,7 @@ var tests = [ test_identifier_keywords, test_getOwnPropertyDescriptor, test_defineProperty, + test_defineProperties, test_property_definitions, test_string_trim, test_global_properties,

1 0

Jacek Caban : jscript: Fix Object.defineProperty return value.
by Alexandre Julliard 21 Aug '19

21 Aug '19

Module: wine Branch: master Commit: 7dab372271f8423e4ff2f6612179eaf5d4572610 URL: https://source.winehq.org/git/wine.git/?a=commit;h=7dab372271f8423e4ff2f661… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Aug 21 12:32:01 2019 +0200 jscript: Fix Object.defineProperty return value. Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/jscript/object.c | 2 ++ dlls/mshtml/tests/es5.js | 5 +++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/jscript/object.c b/dlls/jscript/object.c index 123541e..58db45b 100644 --- a/dlls/jscript/object.c +++ b/dlls/jscript/object.c @@ -500,6 +500,8 @@ static HRESULT Object_defineProperty(script_ctx_t *ctx, vdisp_t *jsthis, WORD fl hres = jsdisp_define_property(obj, name, &prop_desc); release_property_descriptor(&prop_desc); + if(SUCCEEDED(hres) && r) + *r = jsval_obj(jsdisp_addref(obj)); return hres; } diff --git a/dlls/mshtml/tests/es5.js b/dlls/mshtml/tests/es5.js index 11b68e5..7ee8cc1 100644 --- a/dlls/mshtml/tests/es5.js +++ b/dlls/mshtml/tests/es5.js @@ -311,8 +311,9 @@ function test_defineProperty() { ok(false, "expected exception"); } - var obj = new Object(); - Object.defineProperty(obj, "test", {}); + var obj = new Object(), defined; + defined = Object.defineProperty(obj, "test", {}); + ok(defined === obj, "defined != obj"); ok("test" in obj, "test is not in obj"); test_own_data_prop_desc(obj, "test", false, false, false); ok(obj.test === undefined, "obj.test = " + obj.test);

1 0

Rémi Bernon : msctf/tests: Add multi-threaded apartment test for ITfSource_AdviseSink.
by Alexandre Julliard 21 Aug '19

21 Aug '19

Module: wine Branch: master Commit: b9b834b6ba768891c8d6a90c44ff9438e48af012 URL: https://source.winehq.org/git/wine.git/?a=commit;h=b9b834b6ba768891c8d6a90c… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Aug 20 17:46:56 2019 +0200 msctf/tests: Add multi-threaded apartment test for ITfSource_AdviseSink. We expect it to fail because the ITfUIElementSink interface is flagged as "local" and cannot be marshalled, and because ITfThreadMgr is flagged with "threading(apartment)" and cannot be instanciated in the MTA. Signed-off-by: Rémi Bernon <rbernon(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/msctf/tests/inputprocessor.c | 48 +++++++++++++++++++++++++++++++++++++++ 1 file changed, 48 insertions(+) diff --git a/dlls/msctf/tests/inputprocessor.c b/dlls/msctf/tests/inputprocessor.c index d73a3f9..1222085 100644 --- a/dlls/msctf/tests/inputprocessor.c +++ b/dlls/msctf/tests/inputprocessor.c @@ -2511,6 +2511,53 @@ static void test_profile_mgr(void) ITfInputProcessorProfileMgr_Release(ipp_mgr); } +static DWORD WINAPI test_MultiThreadApartment_Thread(void *param) { + ITfThreadMgrEx *thmgr; + ITfSource *source; + DWORD cookie; + HRESULT hr; + + hr = CoInitializeEx(NULL, COINIT_MULTITHREADED); + ok(SUCCEEDED(hr), "Failed to initialize multi-threaded apartment\n"); + + hr = CoCreateInstance(&CLSID_TF_ThreadMgr, NULL, CLSCTX_INPROC_SERVER, &IID_ITfThreadMgrEx, (LPVOID *)&thmgr); + ok(SUCCEEDED(hr), "Failed to create ITfThreadMgrEx instance\n"); + + hr = ITfThreadMgrEx_QueryInterface(thmgr, &IID_ITfSource, (LPVOID *)&source); + ok(SUCCEEDED(hr), "Failed to query ITfSource interface\n"); + + hr = ITfSource_AdviseSink(source, &IID_ITfUIElementSink, (IUnknown*)&TfUIElementSink, &cookie); + ok(hr == REGDB_E_IIDNOTREG /* native */ || hr == E_NOINTERFACE /* wine */, + "Advise ITfUIElementSink should return marshalling failure: %08x\n", hr); + + hr = ITfSource_Release(source); + ok(SUCCEEDED(hr), "Failed to Release source\n"); + + hr = ITfThreadMgrEx_Release(thmgr); + ok(SUCCEEDED(hr), "Failed to Release thread manager\n"); + + CoUninitialize(); + + return 0xdeadcafe; +} + +static void test_MultiThreadApartment(void) +{ + DWORD ret; + HANDLE thread; + + thread = CreateThread(0, 0, test_MultiThreadApartment_Thread, 0, 0, 0); + ok(thread != NULL, "Failed to create test thread\n"); + + ret = WaitForSingleObject(thread, INFINITE); + ok(ret == WAIT_OBJECT_0, "Failed to wait for thread completion\n"); + + GetExitCodeThread(thread, &ret); + ok(ret == 0xdeadcafe, "Thread terminated in an unexpected way\n"); + + CloseHandle(thread); +} + START_TEST(inputprocessor) { if (SUCCEEDED(initialize())) @@ -2540,6 +2587,7 @@ START_TEST(inputprocessor) test_UnregisterCategory(); test_Unregister(); test_profile_mgr(); + test_MultiThreadApartment(); ITextStoreACPSink_Release(ACPSink); ITfDocumentMgr_Release(g_dm);

1 0

Isira Seneviratne : jscript: Replace '%pure-parser' with '%define api.pure '.
by Alexandre Julliard 21 Aug '19

21 Aug '19

Module: wine Branch: master Commit: e0e57b140a02c4a665510cba7fc8c32084c00f68 URL: https://source.winehq.org/git/wine.git/?a=commit;h=e0e57b140a02c4a665510cba… Author: Isira Seneviratne <isirasen96(a)gmail.com> Date: Sun Aug 18 16:47:06 2019 +0530 jscript: Replace '%pure-parser' with '%define api.pure'. Signed-off-by: Isira Seneviratne <isirasen96(a)gmail.com> Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/jscript/cc_parser.y | 2 +- dlls/jscript/parser.y | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/jscript/cc_parser.y b/dlls/jscript/cc_parser.y index 18c1cb1..4bc6260 100644 --- a/dlls/jscript/cc_parser.y +++ b/dlls/jscript/cc_parser.y @@ -30,7 +30,7 @@ WINE_DEFAULT_DEBUG_CHANNEL(jscript); %lex-param { parser_ctx_t *ctx } %parse-param { parser_ctx_t *ctx } -%pure-parser +%define api.pure %start CCExpr %union { diff --git a/dlls/jscript/parser.y b/dlls/jscript/parser.y index 138b6f3..ca83b6f 100644 --- a/dlls/jscript/parser.y +++ b/dlls/jscript/parser.y @@ -141,7 +141,7 @@ static source_elements_t *source_elements_add_statement(source_elements_t*,state %lex-param { parser_ctx_t *ctx } %parse-param { parser_ctx_t *ctx } -%pure-parser +%define api.pure %start Program %union {

1 0

Nikolay Sivov : gdiplus: Add GdipRecordMetafileStreamI().
by Alexandre Julliard 21 Aug '19

21 Aug '19

Module: wine Branch: master Commit: 88207f0c117cc28ae60190954e6ad110efa0e5b1 URL: https://source.winehq.org/git/wine.git/?a=commit;h=88207f0c117cc28ae6019095… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Tue Aug 20 22:48:35 2019 +0300 gdiplus: Add GdipRecordMetafileStreamI(). Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> Signed-off-by: Vincent Povirk <vincent(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/gdiplus/gdiplus.spec | 2 +- dlls/gdiplus/metafile.c | 21 +++++++++++++++++++++ include/gdiplusflat.h | 5 +++-- 3 files changed, 25 insertions(+), 3 deletions(-) diff --git a/dlls/gdiplus/gdiplus.spec b/dlls/gdiplus/gdiplus.spec index ed9d50c..ee28b0e 100644 --- a/dlls/gdiplus/gdiplus.spec +++ b/dlls/gdiplus/gdiplus.spec @@ -451,7 +451,7 @@ 451 stdcall GdipRecordMetafileFileNameI(wstr long long ptr long wstr ptr) 452 stdcall GdipRecordMetafileI(long long ptr long wstr ptr) 453 stdcall GdipRecordMetafileStream(ptr long long ptr long wstr ptr) -454 stub GdipRecordMetafileStreamI +454 stdcall GdipRecordMetafileStreamI(ptr long long ptr long wstr ptr) 455 stdcall GdipReleaseDC(ptr ptr) 456 stdcall GdipRemovePropertyItem(ptr long) 457 stdcall GdipResetClip(ptr) diff --git a/dlls/gdiplus/metafile.c b/dlls/gdiplus/metafile.c index 47d5ac5..a292483 100644 --- a/dlls/gdiplus/metafile.c +++ b/dlls/gdiplus/metafile.c @@ -859,6 +859,27 @@ GpStatus WINGDIPAPI GdipRecordMetafileI(HDC hdc, EmfType type, GDIPCONST GpRect return GdipRecordMetafile(hdc, type, pFrameRectF, frameUnit, desc, metafile); } +GpStatus WINGDIPAPI GdipRecordMetafileStreamI(IStream *stream, HDC hdc, EmfType type, GDIPCONST GpRect *frameRect, + MetafileFrameUnit frameUnit, GDIPCONST WCHAR *desc, GpMetafile **metafile) +{ + GpRectF frameRectF, *pFrameRectF; + + TRACE("(%p %p %d %p %d %p %p)\n", stream, hdc, type, frameRect, frameUnit, desc, metafile); + + if (frameRect) + { + frameRectF.X = frameRect->X; + frameRectF.Y = frameRect->Y; + frameRectF.Width = frameRect->Width; + frameRectF.Height = frameRect->Height; + pFrameRectF = &frameRectF; + } + else + pFrameRectF = NULL; + + return GdipRecordMetafileStream(stream, hdc, type, pFrameRectF, frameUnit, desc, metafile); +} + GpStatus WINGDIPAPI GdipRecordMetafileStream(IStream *stream, HDC hdc, EmfType type, GDIPCONST GpRectF *frameRect, MetafileFrameUnit frameUnit, GDIPCONST WCHAR *desc, GpMetafile **metafile) { diff --git a/include/gdiplusflat.h b/include/gdiplusflat.h index 57fa6c6..1e26c0a 100644 --- a/include/gdiplusflat.h +++ b/include/gdiplusflat.h @@ -284,8 +284,6 @@ GpStatus WINGDIPAPI GdipRecordMetafileFileName(GDIPCONST WCHAR*,HDC,EmfType, GDIPCONST GpRectF*,MetafileFrameUnit,GDIPCONST WCHAR*,GpMetafile**); GpStatus WINGDIPAPI GdipRecordMetafileFileNameI(GDIPCONST WCHAR*,HDC,EmfType, GDIPCONST GpRect*,MetafileFrameUnit,GDIPCONST WCHAR*,GpMetafile**); -GpStatus WINGDIPAPI GdipRecordMetafileI(HDC,EmfType,GDIPCONST GpRect*, - MetafileFrameUnit,GDIPCONST WCHAR*,GpMetafile**); GpStatus WINGDIPAPI GdipReleaseDC(GpGraphics*,HDC); GpStatus WINGDIPAPI GdipResetClip(GpGraphics*); GpStatus WINGDIPAPI GdipResetWorldTransform(GpGraphics*); @@ -549,6 +547,9 @@ GpStatus WINGDIPAPI GdipGetHemfFromMetafile(GpMetafile*,HENHMETAFILE*); GpStatus WINGDIPAPI GdipPlayMetafileRecord(GDIPCONST GpMetafile*,EmfPlusRecordType,UINT,UINT,GDIPCONST BYTE*); GpStatus WINGDIPAPI GdipSetMetafileDownLevelRasterizationLimit(GpMetafile*,UINT); GpStatus WINGDIPAPI GdipRecordMetafile(HDC,EmfType,GDIPCONST GpRectF*,MetafileFrameUnit,GDIPCONST WCHAR*,GpMetafile**); +GpStatus WINGDIPAPI GdipRecordMetafileI(HDC,EmfType,GDIPCONST GpRect*,MetafileFrameUnit,GDIPCONST WCHAR*,GpMetafile**); +GpStatus WINGDIPAPI GdipRecordMetafileStream(IStream*,HDC,EmfType,GDIPCONST GpRectF*,MetafileFrameUnit,GDIPCONST WCHAR *,GpMetafile**); +GpStatus WINGDIPAPI GdipRecordMetafileStreamI(IStream*,HDC,EmfType,GDIPCONST GpRect*,MetafileFrameUnit,GDIPCONST WCHAR *,GpMetafile**); /* MetafileHeader */ GpStatus WINGDIPAPI GdipGetMetafileHeaderFromEmf(HENHMETAFILE,MetafileHeader*);

1 0

Zebediah Figura : widl: Generate a name for the encapsulated union type.
by Alexandre Julliard 21 Aug '19

21 Aug '19

Module: wine Branch: master Commit: 732f67ed38a45911d726df8da5a68710d175ba1d URL: https://source.winehq.org/git/wine.git/?a=commit;h=732f67ed38a45911d726df8d… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Wed Aug 21 11:18:06 2019 -0500 widl: Generate a name for the encapsulated union type. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/widl/parser.y | 3 +-- tools/widl/typetree.c | 2 +- tools/widl/typetree.h | 1 + 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/tools/widl/parser.y b/tools/widl/parser.y index 072e138..8496109 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -82,7 +82,6 @@ static var_t *reg_const(var_t *var); static void push_namespace(const char *name); static void pop_namespace(const char *name); -static char *gen_name(void); static void check_arg_attrs(const var_t *arg); static void check_statements(const statement_list_t *stmts, int is_inside_library); static void check_all_user_types(const statement_list_t *stmts); @@ -2033,7 +2032,7 @@ var_t *find_const(const char *name, int f) return cur->var; } -static char *gen_name(void) +char *gen_name(void) { static const char format[] = "__WIDL_%s_generated_name_%08lX"; static unsigned long n = 0; diff --git a/tools/widl/typetree.c b/tools/widl/typetree.c index 524d005..e9afc6f 100644 --- a/tools/widl/typetree.c +++ b/tools/widl/typetree.c @@ -366,7 +366,7 @@ type_t *type_new_encapsulated_union(char *name, var_t *switch_field, var_t *unio { if (!union_field) union_field = make_var(xstrdup("tagged_union")); - union_field->declspec.type = type_new_nonencapsulated_union(NULL, TRUE, cases); + union_field->declspec.type = type_new_nonencapsulated_union(gen_name(), TRUE, cases); t->details.structure = xmalloc(sizeof(*t->details.structure)); t->details.structure->fields = append_var(NULL, switch_field); diff --git a/tools/widl/typetree.h b/tools/widl/typetree.h index f025f25..e288c57 100644 --- a/tools/widl/typetree.h +++ b/tools/widl/typetree.h @@ -51,6 +51,7 @@ void type_module_define(type_t *module, statement_list_t *stmts); type_t *type_coclass_define(type_t *coclass, ifref_list_t *ifaces); int type_is_equal(const type_t *type1, const type_t *type2); const char *type_get_name(const type_t *type, enum name_type name_type); +char *gen_name(void); /* FIXME: shouldn't need to export this */ type_t *duptype(type_t *t, int dupname);

1 0

Zebediah Figura : widl: Handle encapsulated unions in type libraries.
by Alexandre Julliard 21 Aug '19

21 Aug '19

Module: wine Branch: master Commit: 4f4763ab6b81effe05705eb65c9761814a1402bc URL: https://source.winehq.org/git/wine.git/?a=commit;h=4f4763ab6b81effe05705eb6… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Wed Aug 21 11:18:07 2019 -0500 widl: Handle encapsulated unions in type libraries. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/widl/write_msft.c | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/tools/widl/write_msft.c b/tools/widl/write_msft.c index 420404a..5d5fd89 100644 --- a/tools/widl/write_msft.c +++ b/tools/widl/write_msft.c @@ -976,6 +976,7 @@ static int encode_type( switch (type_get_type(type)) { case TYPE_STRUCT: + case TYPE_ENCAPSULATED_UNION: add_structure_typeinfo(typelib, type); break; case TYPE_INTERFACE: @@ -2119,6 +2120,7 @@ static void add_interface_typeinfo(msft_typelib_t *typelib, type_t *interface) static void add_structure_typeinfo(msft_typelib_t *typelib, type_t *structure) { + var_list_t *fields; int idx = 0; var_t *cur; msft_typeinfo_t *msft_typeinfo; @@ -2130,9 +2132,16 @@ static void add_structure_typeinfo(msft_typelib_t *typelib, type_t *structure) msft_typeinfo = create_msft_typeinfo(typelib, TKIND_RECORD, structure->name, structure->attrs); msft_typeinfo->typeinfo->size = 0; - if (type_struct_get_fields(structure)) - LIST_FOR_EACH_ENTRY( cur, type_struct_get_fields(structure), var_t, entry ) + if (type_get_type(structure) == TYPE_STRUCT) + fields = type_struct_get_fields(structure); + else + fields = type_encapsulated_union_get_fields(structure); + + if (fields) + { + LIST_FOR_EACH_ENTRY( cur, fields, var_t, entry ) add_var_desc(msft_typeinfo, idx++, cur); + } } static void add_enum_typeinfo(msft_typelib_t *typelib, type_t *enumeration) @@ -2320,6 +2329,7 @@ static void add_type_typeinfo(msft_typelib_t *typelib, type_t *type) add_interface_typeinfo(typelib, type); break; case TYPE_STRUCT: + case TYPE_ENCAPSULATED_UNION: add_structure_typeinfo(typelib, type); break; case TYPE_ENUM:

1 0

Zebediah Figura : widl: Write public typedefs into the typelib if they are referenced directly.
by Alexandre Julliard 21 Aug '19

21 Aug '19

Module: wine Branch: master Commit: 96e11d51bae0ef323c32fd0ba1668be0d0283092 URL: https://source.winehq.org/git/wine.git/?a=commit;h=96e11d51bae0ef323c32fd0b… Author: Zebediah Figura <z.figura12(a)gmail.com> Date: Wed Aug 21 11:18:08 2019 -0500 widl: Write public typedefs into the typelib if they are referenced directly. I.e. write "apple_t" into the typelib given the following snippet: typedef [public] struct { ... } apple_t; library { interface IFoo { HRESULT a(apple_t arg); } } Based on a patch by Richard Pospesel. Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/widl/write_msft.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tools/widl/write_msft.c b/tools/widl/write_msft.c index 5d5fd89..e972a26 100644 --- a/tools/widl/write_msft.c +++ b/tools/widl/write_msft.c @@ -763,6 +763,7 @@ static void add_enum_typeinfo(msft_typelib_t *typelib, type_t *enumeration); static void add_union_typeinfo(msft_typelib_t *typelib, type_t *tunion); static void add_coclass_typeinfo(msft_typelib_t *typelib, type_t *cls); static void add_dispinterface_typeinfo(msft_typelib_t *typelib, type_t *dispinterface); +static void add_typedef_typeinfo(msft_typelib_t *typelib, type_t *dispinterface); /**************************************************************************** @@ -973,7 +974,7 @@ static int encode_type( chat("encode_type: VT_USERDEFINED - adding new type %s, real type %d\n", type->name, type_get_type(type)); - switch (type_get_type(type)) + switch (type_get_type_detect_alias(type)) { case TYPE_STRUCT: case TYPE_ENCAPSULATED_UNION: @@ -991,6 +992,9 @@ static int encode_type( case TYPE_COCLASS: add_coclass_typeinfo(typelib, type); break; + case TYPE_ALIAS: + add_typedef_typeinfo(typelib, type); + break; default: error("encode_type: VT_USERDEFINED - unhandled type %d\n", type_get_type(type));

1 0

Richard Pospesel : widl: Add the typedef to the type library if the aliasee is an anonymous UDT.
by Alexandre Julliard 21 Aug '19

21 Aug '19

Module: wine Branch: master Commit: 4b1dfa60371a69f978455edc52a2a1cfef33eb4f URL: https://source.winehq.org/git/wine.git/?a=commit;h=4b1dfa60371a69f978455edc… Author: Richard Pospesel <richard(a)torproject.org> Date: Wed Aug 21 11:18:09 2019 -0500 widl: Add the typedef to the type library if the aliasee is an anonymous UDT. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=47050 Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- tools/widl/parser.y | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tools/widl/parser.y b/tools/widl/parser.y index 8496109..59875a5 100644 --- a/tools/widl/parser.y +++ b/tools/widl/parser.y @@ -1895,7 +1895,11 @@ static type_t *reg_typedefs(decl_spec_t *decl_spec, declarator_list_t *decls, at type_get_type_detect_alias(type) == TYPE_ENCAPSULATED_UNION) { if (!type->name) + { type->name = gen_name(); + if (!is_attr(attrs, ATTR_PUBLIC)) + attrs = append_attr(attrs, make_attr(ATTR_PUBLIC)); + } /* replace existing attributes when generating a typelib */ if (do_typelib)

1 0