- wine-devel - winehq.org

Re: [PATCH] midimap: Add warning for user in case midi won't work
by Matteo Bruni 29 Sep '17

29 Sep '17

2017-09-23 22:52 GMT+02:00 Fabian Maurer <dark.shadow4(a)web.de>: > When no midi port or only the midi-through port > is found, we can inform the user that midi > won't work as expected. > > Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> > --- > dlls/midimap/midimap.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/dlls/midimap/midimap.c b/dlls/midimap/midimap.c > index 64b4dc8546..c80834515b 100644 > --- a/dlls/midimap/midimap.c > +++ b/dlls/midimap/midimap.c > @@ -553,6 +553,7 @@ static LRESULT MIDIMAP_drvOpen(void) > { > MIDIOUTCAPSW moc; > unsigned dev, i; > + WCHAR throughportW[] = {'M','i','d','i',' ','T','h','r','o','u','g','h',0}; > > if (midiOutPorts) > return 0; > @@ -578,6 +579,13 @@ static LRESULT MIDIMAP_drvOpen(void) > } > } > > + if(numMidiOutPorts == 0) > + MESSAGE("Midi sound output won't work - No midi port found.\n"); > + else if(numMidiOutPorts == 1 && midiOutPorts[0].loaded == -1) > + MESSAGE("Midi sound output won't work - No working midi port found.\n"); > + else if(numMidiOutPorts == 1 && strncmpW(midiOutPorts[0].name, throughportW, strlenW(throughportW)) == 0) > + MESSAGE("Midi sound output probably won't work - Only 'midi through' port found. Make sure your setup is correct.\n"); > + > return 1; > } Since you have to resend anyway... The string comparison doesn't look correct to me, I think that breaks if the midi port name has a prefix matching the throughportW string. I guess it's not going to be a problem in practice but it seems better to do it right anyway. There should be a space between 'if' and the '(', like elsewhere in the function and source file.

2 3

Re: [PATCH 3/3] mshtml/tests: Run events tests in IE9 mode.
by Alexandre Julliard 29 Sep '17

29 Sep '17

Jacek Caban <jacek(a)codeweavers.com> writes: > Signed-off-by: Jacek Caban <jacek(a)codeweavers.com> > --- > dlls/mshtml/tests/events.c | 132 > +++++++++++++++++++++++++++++++++++---------- > 1 file changed, 103 insertions(+), 29 deletions(-) It doesn't work here: ../../../tools/runtest -q -P wine -T ../../.. -M mshtml.dll -p mshtml_test.exe.so events && touch events.ok events.c:917: Test failed: pdp->cArgs = 1 events.c:954: Test failed: pdp->cArgs = 1 events.c:903: Test failed: pdp->cArgs = 1 events.c:917: Test failed: pdp->cArgs = 1 events.c:954: Test failed: pdp->cArgs = 1 events.c:903: Test failed: pdp->cArgs = 1 events.c:917: Test failed: pdp->cArgs = 1 events.c:954: Test failed: pdp->cArgs = 1 events.c:903: Test failed: pdp->cArgs = 1 events.c:917: Test failed: pdp->cArgs = 1 events.c:954: Test failed: pdp->cArgs = 1 events.c:903: Test failed: pdp->cArgs = 1 events.c:954: Test failed: pdp->cArgs = 1 events.c:903: Test failed: pdp->cArgs = 1 events.c:1242: Test failed: pdp->cArgs = 1 events.c:1218: Test failed: pdp->cArgs = 1 events.c:1242: Test failed: pdp->cArgs = 1 events.c:1218: Test failed: pdp->cArgs = 1 events.c:1242: Test failed: pdp->cArgs = 1 events.c:1074: Test failed: pdp->cArgs = 1 events.c:1060: Test failed: pdp->cArgs = 1 events.c:1085: Test failed: pdp->cArgs = 1 events.c:1074: Test failed: pdp->cArgs = 1 events.c:1085: Test failed: pdp->cArgs = 1 events.c:1074: Test failed: pdp->cArgs = 1 events.c:1074: Test failed: pdp->cArgs = 1 events.c:1074: Test failed: pdp->cArgs = 1 events.c:1151: Test failed: pdp->cArgs = 1 events.c:1074: Test failed: pdp->cArgs = 1 events.c:1060: Test failed: pdp->cArgs = 1 events.c:1151: Test failed: pdp->cArgs = 1 events.c:1074: Test failed: pdp->cArgs = 1 events.c:1151: Test failed: pdp->cArgs = 1 events.c:1074: Test failed: pdp->cArgs = 1 events.c:1151: Test failed: pdp->cArgs = 1 events.c:1074: Test failed: pdp->cArgs = 1 events.c:1191: Test failed: pdp->cArgs = 1 Makefile:267: recipe for target 'events.ok' failed make[1]: *** [events.ok] Error 37 -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: [PATCH 1/3] mshtml: Inherit document mode from parent document.
by Marvin 29 Sep '17

29 Sep '17

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=33334 Your paranoid android. === w7u (32 bit htmldoc) === htmldoc.c:2682: Test failed: unexpected call GetOverrideKeyPath htmldoc.c:5092: Test failed: unexpected call {d9befc84-f21e-4166-874b-a8d3010dc764}

1 0

Re: mshtml: Inherit document mode from parent document.
by Gijs Vermeulen 29 Sep '17

29 Sep '17

Hi, 2017-09-28 19:50 GMT+02:00 Jacek Caban <jacek(a)codeweavers.com>: > > + TRACE("parnet mode %u\n", parent_mode); > > There seems to be a typo here. Sincerely, Gijs

2 1

Re: [v7 1/5] server: Named pipe query SD from pipe object
by Jacek Caban 29 Sep '17

29 Sep '17

Hi Jonathan, Your tests still don't really show intended behaviour. Your calls that set security descriptor use new SD that has the same tested values as before the call, then what's the point in setting it? Anyway, I cleaned it up and changed a bit to see how it's really supposed to work. See the attachment. It looks like client and served should indeed share security descriptor, but pipe object should not, so that part is wrong. Thanks, Jacek

2 2

Re: mshtml: Inherit document mode from parent document.
by Marvin 28 Sep '17

28 Sep '17

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=33325 Your paranoid android. === w7u (32 bit htmldoc) === htmldoc.c:2682: Test failed: unexpected call GetOverrideKeyPath htmldoc.c:5092: Test failed: unexpected call {d9befc84-f21e-4166-874b-a8d3010dc764}

1 0

Re: [PATCH] midimap: Add warning for user in case midi won't work
by Alexandre Julliard 28 Sep '17

28 Sep '17

Fabian Maurer <dark.shadow4(a)web.de> writes: > When no midi port or only the midi-through port > is found, we can inform the user that midi > won't work as expected. This should probably use the winediag debug channel. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH v3 2/3] hlink/tests: Add tests for navigating from a string reference.
by Jacek Caban 28 Sep '17

28 Sep '17

Hi Zebediah, This test leaves iexplore instance running on Wine. Jacek

2 1

Re: [PATCH 2/9] sapi: Implement SpDataKey::SetKey().
by Marvin 28 Sep '17

28 Sep '17

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=33314 Your paranoid android. === build (build) === Recreation of tests/Makefile failed

2 1

Re: [PATCH 2/2] fonts: Increase the Em size to fix a build failure with freetype 2.8.1
by Dmitry Timoshkov 28 Sep '17

28 Sep '17

Michael Stefaniuc <mstefani(a)winehq.org> wrote: > -UnderlinePosition: -100 > -UnderlineWidth: 50 > -Ascent: 11 > -Descent: 2 > +UnderlinePosition: -15753.8 > +UnderlineWidth: 7876.92 > +Ascent: 1733 > +Descent: 315 > +InvalidEm: 0 > LayerCount: 2 > Layer: 0 0 "Back" 1 > Layer: 1 0 "Fore" 0 > @@ -21,7 +22,7 @@ OS2_UseTypoMetrics: 0 > PfmFamily: 49 > TTFWeight: 400 > TTFWidth: 5 > -LineGap: 1 > +LineGap: 158 > VLineGap: 0 > Panose: 2 0 6 9 0 0 0 0 0 0 > OS2TypoAscent: 0 > @@ -56,770 +57,770 @@ BeginChars: 65536 498 > > StartChar: space > Encoding: 32 32 0 > -Width: 8 > +Width: 1260 > Flags: W > LayerCount: 2 > EndChar > > StartChar: exclam > Encoding: 33 33 1 > -Width: 8 > +Width: 1260 > Flags: W > LayerCount: 2 > EndChar Did you test that after this patch font and glyph metrics still stay the same? -- Dmitry.

2 3

Re: [PATCH 9/9] sapi: Add a partial implementation of SpObjectTokenCategory::EnumTokens().
by Marvin 28 Sep '17

28 Sep '17

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=33316 Your paranoid android. === build (build) === Recreation of tests/Makefile failed

1 0

Re: [PATCH 6/9] sapi: Add a partial implementation of SpObjectTokenEnum::Next().
by Marvin 28 Sep '17

28 Sep '17

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=33315 Your paranoid android. === build (build) === Recreation of tests/Makefile failed

1 0

WineTest does not run the vcomp tests
by Francois Gouget 28 Sep '17

28 Sep '17

So WineTest is not running the vcomp tests. What happens is this: * It parses the vcomp_test.exe filename and concludes that this tests the vcomp dll. * It does a LoadLibraryExA("vcomp.dll", NULL, LOAD_LIBRARY_AS_DATAFILE) which fails (as expected) and concludes that the dll is missing: vcomp=dll is missing * Since the dll is missing WineTest skips the vcomp tests. According to Dan Kegel this is a manifest / activation context issue and he sent a patch for it [1]. However the patch was for the TestBot [2], not WineTest, and as far as I can tell it's not working. Furthermore WineTest already has activation context code that seems equivalent to Dan Kegel's patch. Does anyone know how to fix WineTest? [1] https://bugs.winehq.org/show_bug.cgi?id=31609 https://www.winehq.org/pipermail/wine-patches/2012-September/117631.html [2] Paradoxically the TestBot has no problem running the vcomp tests now that they dynamically load the vcomp dll. That's because the TestBot makes no assumption based on the executable filename since one can upload test binaries that don't follow any naming scheme. So now the test dynamically loads vcomp.dll the TestBot cannot check for it. -- Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/ May your Tongue stick to the Roof of your Mouth with the Force of a Thousand Caramels.

2 2

Re: user.exe16: Support Windows 1.x and 2.x Menus
by Alexandre Julliard 28 Sep '17

28 Sep '17

Martin Payne <development(a)martinpayne.me.uk> writes: > Added support for Windows 1.x / 2.x menu resources to > LoadMenuIndirect16. The format of these are slightly different from > Windows 3.x menus which causes them to fail to load. Where did you find an app that needs this? -- Alexandre Julliard julliard(a)winehq.org

2 3

Re: [PATCH 6/6] shell32: Implement FolderItem::Name() property getter
by Marvin 27 Sep '17

27 Sep '17

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=33296 Your paranoid android. === wxppro (32 bit shelldispatch) === shelldispatch.c:635: Test failed: file_defs[1]: unexpected name L"01-empty".

2 1

Re: [PATCH] ole32: CreateDataCache should return a COleCache object by default.
by Huw Davies 26 Sep '17

26 Sep '17

On Tue, Sep 26, 2017 at 01:00:08AM +0000, Alistair Leslie-Hughes wrote: > Signed-off-by: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> > --- > dlls/ole32/datacache.c | 5 ++++- > dlls/ole32/tests/ole2.c | 10 ++++++++-- > 2 files changed, 12 insertions(+), 3 deletions(-) > > diff --git a/dlls/ole32/datacache.c b/dlls/ole32/datacache.c > index a95d0fb..c563118 100644 > --- a/dlls/ole32/datacache.c > +++ b/dlls/ole32/datacache.c > @@ -2760,7 +2760,10 @@ HRESULT WINAPI CreateDataCache( > if (newCache == 0) > return E_OUTOFMEMORY; > > - hr = IUnknown_QueryInterface(&newCache->IUnknown_inner, riid, ppvObj); > + if ( !pUnkOuter && IsEqualIID(&IID_IUnknown, riid) ) > + hr = IUnknown_QueryInterface(&newCache->IUnknown_inner, &IID_IOleCache2, ppvObj); > + else > + hr = IUnknown_QueryInterface(&newCache->IUnknown_inner, riid, ppvObj); > IUnknown_Release(&newCache->IUnknown_inner); But now if you call QI(IID_IUnknown) on this interface you'll get back something different, which is wrong. You'll need to do some more restructing to make this correct. Btw, is there a bug that depends on this? > diff --git a/dlls/ole32/tests/ole2.c b/dlls/ole32/tests/ole2.c > index b30f63b..39b161c 100644 > --- a/dlls/ole32/tests/ole2.c > +++ b/dlls/ole32/tests/ole2.c > @@ -1575,6 +1575,7 @@ static void test_data_cache(void) > char szSystemDir[MAX_PATH]; > WCHAR wszPath[MAX_PATH]; > static const WCHAR wszShell32[] = {'\\','s','h','e','l','l','3','2','.','d','l','l',0}; > + DWORD connection; > > static const struct expected_method methods_cacheinitnew[] = > { > @@ -1651,14 +1652,19 @@ static void test_data_cache(void) > ok(hr == S_OK, "got 0x%08x\n", hr); > hr = IUnknown_QueryInterface(unk, &IID_IOleCache2, (void**)&pOleCache); > ok(hr == S_OK, "got 0x%08x\n", hr); > -todo_wine { > ok(unk == (IUnknown*)olecache, "got %p, expected %p\n", olecache, unk); > ok(unk == (IUnknown*)pOleCache, "got %p, expected %p\n", pOleCache, unk); > -} > + > IOleCache2_Release(pOleCache); > IOleCache_Release(olecache); > IUnknown_Release(unk); > > + hr = CreateDataCache(NULL, &CLSID_NULL, &IID_IUnknown, (void**)&olecache); > + ok(hr == S_OK, "got 0x%08x\n", hr); > + hr = IOleCache_Cache(olecache, &fmtetc, ADVF_PRIMEFIRST | ADVFCACHE_NOHANDLER, &connection); > + ok(hr == S_OK, "got 0x%08x\n", hr); > + IOleCache_Release(olecache); > + A better test would be to call QI(IID_IOleCache) on the returned IUnknown* and show that it's the same. You could also QI(IID_IUnknown) to test for my comment above. Huw.

2 1

Re: [PATCH 1/2] gdi32: Remove runtime check for FT_Set_Charmap()
by Huw Davies 26 Sep '17

26 Sep '17

On Mon, Sep 25, 2017 at 05:38:53PM +0300, Nikolay Sivov wrote: > Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> > --- > dlls/gdi32/freetype.c | 79 ++++++++++++++++++++++++--------------------------- > 1 file changed, 37 insertions(+), 42 deletions(-) I've resent this series removing pFT_Select_Charmap since it's now not used. Thanks, Huw.

1 0

Re: [PATCH 2/5] shell32: Use shellfolder API to construct FolderItems
by Alexandre Julliard 25 Sep '17

25 Sep '17

Nikolay Sivov <nsivov(a)codeweavers.com> writes: > Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> > --- > dlls/shell32/shelldispatch.c | 195 ++++++++++++++++++++++--------------- > dlls/shell32/tests/shelldispatch.c | 3 + > 2 files changed, 118 insertions(+), 80 deletions(-) This breaks the tests here: ../../../tools/runtest -q -P wine -T ../../.. -M shell32.dll -p shell32_test.exe.so shelldispatch && touch shelldispatch.ok shelldispatch.c:615: Test failed: file_defs[0]: expected L"C:\\users\\julliard\\Temp\\winetest\\00-Myfolder", got L"C:\\users\\julliard\\Temp\\winetest\\01-empty.bin" shelldispatch.c:681: Test failed: file_defs[0]: expected L"C:\\users\\julliard\\Temp\\winetest\\00-Myfolder", got L"C:\\users\\julliard\\Temp\\winetest\\01-empty.bin" shelldispatch.c:615: Test failed: file_defs[1]: expected L"C:\\users\\julliard\\Temp\\winetest\\01-empty.bin", got L"C:\\users\\julliard\\Temp\\winetest\\00-Myfolder" shelldispatch.c:681: Test failed: file_defs[1]: expected L"C:\\users\\julliard\\Temp\\winetest\\01-empty.bin", got L"C:\\users\\julliard\\Temp\\winetest\\00-Myfolder" Makefile:515: recipe for target 'shelldispatch.ok' failed make: *** [shelldispatch.ok] Error 4 -- Alexandre Julliard julliard(a)winehq.org

1 0

[PATCH] dwmapi: Update spec file.
by Andrey Gusev 25 Sep '17

25 Sep '17

> Why are you removing these? I guess it's not so important to leave these.

1 0

Re: [PATCH 2/2] shell32/tests: Check IDispatch for NULL before using its functions
by Nikolay Sivov 25 Sep '17

25 Sep '17

On 25.09.2017 21:34, Fabian Maurer wrote: > Fixes Bug 43749 > > Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> > --- > dlls/shell32/tests/shelldispatch.c | 14 ++++++++++---- > 1 file changed, 10 insertions(+), 4 deletions(-) > > diff --git a/dlls/shell32/tests/shelldispatch.c b/dlls/shell32/tests/shelldispatch.c > index 7b5afa0c94..b31497b557 100644 > --- a/dlls/shell32/tests/shelldispatch.c > +++ b/dlls/shell32/tests/shelldispatch.c > @@ -134,13 +134,15 @@ static void test_namespace(void) > r = IShellDispatch_get_Application(sd, &disp); > ok(r == S_OK, "Failed to get application pointer, hr %#x.\n", r); > ok(disp == (IDispatch *)sd, "Unexpected application pointer %p.\n", disp); > - IDispatch_Release(disp); > + if(disp) > + IDispatch_Release(disp); > > disp = NULL; > r = IShellDispatch_get_Parent(sd, &disp); > ok(r == S_OK, "Failed to get Shell object parent, hr %#x.\n", r); > ok(disp == (IDispatch *)sd, "Unexpected parent pointer %p.\n", disp); > - IDispatch_Release(disp); > + if(disp) > + IDispatch_Release(disp); > > VariantInit(&var); > folder = (void*)0xdeadbeef; > @@ -502,7 +504,8 @@ static void test_items(void) > r = FolderItem_get_Parent(item, &disp); > ok(r == S_OK, "Failed to get parent pointer, hr %#x.\n", r); > ok(disp == (IDispatch *)folder, "Unexpected parent pointer %p.\n", disp); > - IDispatch_Release(disp); > + if(disp) > + IDispatch_Release(disp); > > if (item) FolderItem_Release(item); > VariantClear(&var); > @@ -1036,6 +1039,10 @@ todo_wine { > ok(disp == NULL, "got %p\n", disp); > ok(ret == 0, "got %d\n", ret); > } > + else if (disp == NULL) { > + ok(FALSE, "disp is NULL\n"); > + skip("disp is NULL\n"); > + } > else { > static const IID *browser_riids[] = { > &IID_IWebBrowser2, > @@ -1055,7 +1062,6 @@ todo_wine { > IShellView *sv; > IUnknown *unk; > > - ok(disp != NULL, "got %p\n", disp); > ok(ret != HandleToUlong(hwnd), "got %d\n", ret); > > /* IDispatch-related tests */ > So what happens exactly that makes get_Parent() and get_Application() fail? I don't think workarounds like that are useful without understanding the problem.

2 1

Re: [PATCH] dwmapi: Update spec file.
by Alexandre Julliard 25 Sep '17

25 Sep '17

Andrey Gusev <andrey.goosev(a)gmail.com> writes: > @@ -1,29 +1,10 @@ > -100 stub @ > -101 stub @ > -102 stdcall DwmEnableComposition (long) > -103 stub @ > -104 stub @ > -105 stub @ > -106 stub @ > -107 stub @ > -108 stub @ > -109 stub @ > -110 stub @ > -111 stub @ > -112 stub @ > -113 stub @ > - > -115 stub @ > -116 stub @ > -117 stub @ > -118 stub @ > -119 stub @ > -120 stub @ Why are you removing these? Also in general new stubs should be commented out. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH] gdi32: Pick first charmap if none was selected using preferred encoding
by Huw Davies 25 Sep '17

25 Sep '17

On Mon, Sep 25, 2017 at 02:21:02PM +0300, Nikolay Sivov wrote: > FreeType 2.8.1 has similar logic removed from WinFNT module, > that always pre-selected first charmap. > > Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> > --- > > For https://bugs.winehq.org/show_bug.cgi?id=43715 > > dlls/gdi32/freetype.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/dlls/gdi32/freetype.c b/dlls/gdi32/freetype.c > index 13edef6601..bf566f382c 100644 > --- a/dlls/gdi32/freetype.c > +++ b/dlls/gdi32/freetype.c > @@ -5150,6 +5150,10 @@ static FT_Encoding pick_charmap( FT_Face face, int charset ) > if (select_charmap( face, *encs )) break; > encs++; > } > + > + if (!face->charmap && face->num_charmaps) > + face->charmap = face->charmaps[0]; > + Manipulating face->charmap directly doesn't look right, you probably want to use FT_Set_Charmap(). Huw.

1 0

Re: [PATCH 1/2] msvcrt: Introduce vfprintf helper function.
by Piotr Caban 25 Sep '17

25 Sep '17

Hi, On 09/25/17 01:24, Gijs Vermeulen wrote: > -int CDECL MSVCRT_vfprintf(MSVCRT_FILE* file, const char *format, __ms_va_list valist) > +static int vfprintf_helper(MSVCRT_FILE* file, const char *format, __ms_va_list valist, > + MSVCRT__locale_t locale, BOOL secure) > { > BOOL tmp_buf; > int ret; > > + if(secure) > + if(!MSVCRT_CHECK_PMT(file != NULL)) return -1; > + > MSVCRT__lock_file(file); > tmp_buf = add_std_buffer(file); > - ret = pf_printf_a(puts_clbk_file_a, file, format, NULL, 0, arg_clbk_valist, NULL, &valist); > + > + if(!secure) > + ret = pf_printf_a(puts_clbk_file_a, file, format, NULL, 0, arg_clbk_valist, NULL, &valist); > + else > + { > + if(!locale) > + ret = pf_printf_a(puts_clbk_file_a, file, format, NULL, > + MSVCRT_PRINTF_INVOKE_INVALID_PARAM_HANDLER, arg_clbk_valist, NULL, &valist); > + else > + ret = pf_printf_a(puts_clbk_file_a, file, format, locale, 0, arg_clbk_valist, NULL, &valist); > + } > + > if(tmp_buf) remove_std_buffer(file); > MSVCRT__unlock_file(file); > > return ret; > } This helper changes current functions behavior. Thanks, Piotr

1 0

Re: [v2 PATCH] msvcrt: Impement clearerr_s
by Piotr Caban 25 Sep '17

25 Sep '17

Hi Alistair, On 09/22/17 11:50, Alistair Leslie-Hughes wrote: > +int CDECL MSVCRT_clearerr_s(MSVCRT_FILE* file) > +{ > + TRACE(":file (%p) fd (%d)\n",file,file->_file); > + > + if (!MSVCRT_CHECK_PMT(file != NULL)) return MSVCRT_EINVAL; You're checking if file != NULL after using it in TRACE message. Thanks, Piotr

1 0

Re: [PATCH 1/2] include: Added stgprop.h
by Dmitry Timoshkov 25 Sep '17

25 Sep '17

Nikolay Sivov <nsivov(a)codeweavers.com> wrote: > Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> > --- > include/ntquery.h | 25 +------------------------ > include/stgprop.h | 43 +++++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 44 insertions(+), 24 deletions(-) > create mode 100644 include/stgprop.h > --- /dev/null > +++ b/include/stgprop.h > @@ -0,0 +1,43 @@ > +/* > + * Copyright 2017 Nikolay Sivov Shouldn't you convey the copyright information from ntquery.h? -- Dmitry.

1 0

Re: [v6 4/5] server: all objects must have a SD
by Jacek Caban 22 Sep '17

22 Sep '17

Hi Jonathan, If you're changing behaviour of about all objects, it needs tests for more object types first. Jacek

2 1

Re: Fabian Maurer : evr/tests: Add tests.
by Fabian Maurer 22 Sep '17

22 Sep '17

On Donnerstag, 21. September 2017 22:01:00 CEST Alexandre Julliard wrote: > Module: wine > Branch: master > Commit: 98923665105334ca6aaf39a756b62da606ecc861 > URL: > http://source.winehq.org/git/wine.git/?a=commit;h=98923665105334ca6aaf39a75 > 6b62da606ecc861 > > Author: Fabian Maurer <dark.shadow4(a)web.de> > Date: Wed Sep 20 14:03:36 2017 +0200 > > evr/tests: Add tests. > > Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> > Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> Wasn't this rejected? Regards, Fabian Maurer

2 2

Re: vbscript: Improve parsing of separators in loops and switches.
by Jacek Caban 22 Sep '17

22 Sep '17

Hi Dmitry, Thanks for the patch, it generally looks good. I have a few comments bellow. On 20.09.2017 05:48, Dmitry Kislyuk wrote: > > +StSeps > + : StSep > + | StSep StSeps Since you're changing all uses of StSep, I think it would make more sense to change StSep itself to accept multiple separator. > x = 0 > -Do While x < 2 : x = x + 1 > +Do While x < 2 : > +x = x + 1 > Loop > Call ok(x = 2, "x not equal to 2") Please don't change existing tests like that. We want previous version tested as well. Add new ones instead. > @@ -511,7 +533,8 @@ Call collectionObj.reset() > y = 0 > x = 10 > z = 0 > -for each x in collectionObj : z = z + 2 > +for each x in collectionObj : > + :z = z + 2 Same here. Thanks, Jacek

1 0

Re: [PATCH 1/7] include: Convert dxgitype.h to IDL.
by Alistair Leslie-Hughes 22 Sep '17

22 Sep '17

Hi Henri, On 22/09/17 06:29, Henri Verbeet wrote: > Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> > --- > include/dxgi.idl | 2 +- > include/{dxgitype.h => dxgitype.idl} | 83 ++++++++++++++++++++---------------- You forgot to update the Makefile.in to reflect the name change. (same goes for rest of the patchset) Best Regards Alistair.

2 1

Re: [PATCH] msvcrt: Implement clearerr_s
by Nikolay Sivov 22 Sep '17

22 Sep '17

Docs say it should invoke invalid parameter handler and set error number.

1 0

Re: [PATCH v2 3/3] wined3d: Advertise AUTOGENMIPMAP only for renderable and filterable formats.
by Henri Verbeet 21 Sep '17

21 Sep '17

On 21 September 2017 at 01:07, Matteo Bruni <mbruni(a)codeweavers.com> wrote: > + || (format->flags[gl_type] & (WINED3DFMT_FLAG_RENDERTARGET | WINED3DFMT_FLAG_FILTERING)) > + != (WINED3DFMT_FLAG_RENDERTARGET | WINED3DFMT_FLAG_FILTERING))) As an aside, that could also be written as "~format->flags[gl_type] & (WINED3DFMT_FLAG_RENDERTARGET | WINED3DFMT_FLAG_FILTERING)"

1 0

Re: [PATCH] wine/test.h: Increment correct counter for tests in todo blocks
by Nikolay Sivov 21 Sep '17

21 Sep '17

On 21.09.2017 19:11, Fabian Maurer wrote: > Tests succeeding in todo blocks are counted as failures, > and failures as success. This patch fixes that. > > Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> > --- > include/wine/test.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/wine/test.h b/include/wine/test.h > index 4ec66d5391..f0ee617d5f 100644 > --- a/include/wine/test.h > +++ b/include/wine/test.h > @@ -311,7 +311,7 @@ int winetest_vok( int condition, const char *msg, __winetest_va_list args ) > printf( "%s:%d: Test succeeded inside todo block: ", > data->current_file, data->current_line ); > vprintf(msg, args); > - InterlockedIncrement(&todo_failures); > + InterlockedIncrement(&todo_successes); > return 0; > } > else > @@ -322,7 +322,7 @@ int winetest_vok( int condition, const char *msg, __winetest_va_list args ) > data->current_file, data->current_line ); > vprintf(msg, args); > } > - InterlockedIncrement(&todo_successes); > + InterlockedIncrement(&todo_failures); > return 1; > } > } > This is correct. If test passes but is marked as todo it is a failure.

2 2

Re: [v5 1/3] evr: Add tests
by Alexandre Julliard 21 Sep '17

21 Sep '17

Fabian Maurer <dark.shadow4(a)web.de> writes: > +static void test_evr_filter_aggregations(void) > +{ > + const IID * iids[] = { > + &IID_IAMCertifiedOutputProtection, &IID_IAMFilterMiscFlags, &IID_IBaseFilter, > + &IID_IKsPropertySet, &IID_IMediaEventSink, &IID_IMediaSeeking, &IID_IQualityControl, > + &IID_IQualProp, &IID_IEVRFilterConfig, &IID_IMFGetService, &IID_IMFVideoPositionMapper, > + &IID_IMFVideoRenderer, &IID_IQualityControl > + }; > + int i; > + > + if(!strcmp(winetest_platform, "wine")) > + { > + skip("Not supported yet.\n"); > + return; > + } I don't think it's useful to have a test that doesn't test anything. You should mark the tests as todo but still run them. If that's not possible, it should wait until we have something to test. -- Alexandre Julliard julliard(a)winehq.org

2 2

Re: [PATCH 3/3] msi: Defer the appropriate standard actions to the install script.
by Hans Leidekker 21 Sep '17

21 Sep '17

On Wed, 2017-09-20 at 23:53 -0500, Zebediah Figura wrote: > diff --git a/dlls/msi/action.c b/dlls/msi/action.c > index 915ff173e9..b5ed7c75a8 100644 > --- a/dlls/msi/action.c > +++ b/dlls/msi/action.c > @@ -7764,6 +7764,7 @@ static UINT ACTION_InstallSFPCatalogFile( MSIPACKAGE *package ) > > static const struct > { > + BOOL deferred; > const WCHAR *action; > const UINT description; > const UINT template; > @@ -7772,83 +7773,83 @@ static const struct > } > StandardActions[] = > { [...] > + { 0, szAllocateRegistrySpace, IDS_DESC_ALLOCATEREGISTRYSPACE, IDS_TEMP_ALLOCATEREGISTRYSPACE, ACTION_AllocateRegistrySpace, NULL }, > + { 0, szAppSearch, IDS_DESC_APPSEARCH, IDS_TEMP_APPSEARCH, ACTION_AppSearch, NULL }, > + { 0, szBindImage, IDS_DESC_BINDIMAGE, IDS_TEMP_BINDIMAGE, ACTION_BindImage, NULL }, > + { 0, szCCPSearch, IDS_DESC_CCPSEARCH, 0, ACTION_CCPSearch, NULL }, > + { 0, szCostFinalize, IDS_DESC_COSTFINALIZE, 0, ACTION_CostFinalize, NULL }, > + { 0, szCostInitialize, IDS_DESC_COSTINITIALIZE, 0, ACTION_CostInitialize, NULL }, > + { 1, szCreateFolders, IDS_DESC_CREATEFOLDERS, IDS_TEMP_CREATEFOLDERS, ACTION_CreateFolders, szRemoveFolders }, > + { 1, szCreateShortcuts, IDS_DESC_CREATESHORTCUTS, IDS_TEMP_CREATESHORTCUTS, ACTION_CreateShortcuts, szRemoveShortcuts }, > + { 1, szDeleteServices, IDS_DESC_DELETESERVICES, IDS_TEMP_DELETESERVICES, ACTION_DeleteServices, szInstallServices }, > + { 0, szDisableRollback, 0, 0, ACTION_DisableRollback, NULL }, > + { 1, szDuplicateFiles, IDS_DESC_DUPLICATEFILES, IDS_TEMP_DUPLICATEFILES, ACTION_DuplicateFiles, szRemoveDuplicateFiles }, > + { 0, szExecuteAction, 0, 0, ACTION_ExecuteAction, NULL }, > + { 0, szFileCost, IDS_DESC_FILECOST, 0, ACTION_FileCost, NULL }, > + { 0, szFindRelatedProducts, IDS_DESC_FINDRELATEDPRODUCTS, IDS_TEMP_FINDRELATEDPRODUCTS, ACTION_FindRelatedProducts, NULL }, > + { 0, szForceReboot, 0, 0, ACTION_ForceReboot, NULL }, > + { 0, szInstallAdminPackage, IDS_DESC_INSTALLADMINPACKAGE, IDS_TEMP_INSTALLADMINPACKAGE, ACTION_InstallAdminPackage, NULL }, > + { 0, szInstallExecute, 0, 0, ACTION_InstallExecute, NULL }, > + { 0, szInstallExecuteAgain, 0, 0, ACTION_InstallExecute, NULL }, > + { 1, szInstallFiles, IDS_DESC_INSTALLFILES, IDS_TEMP_INSTALLFILES, ACTION_InstallFiles, szRemoveFiles }, > + { 0, szInstallFinalize, 0, 0, ACTION_InstallFinalize, NULL }, > + { 0, szInstallInitialize, 0, 0, ACTION_InstallInitialize, NULL }, > + { 1, szInstallODBC, IDS_DESC_INSTALLODBC, 0, ACTION_InstallODBC, szRemoveODBC }, > + { 1, szInstallServices, IDS_DESC_INSTALLSERVICES, IDS_TEMP_INSTALLSERVICES, ACTION_InstallServices, szDeleteServices }, > + { 0, szInstallSFPCatalogFile, IDS_DESC_INSTALLSFPCATALOGFILE, IDS_TEMP_INSTALLSFPCATALOGFILE, ACTION_InstallSFPCatalogFile, NULL }, > + { 0, szInstallValidate, IDS_DESC_INSTALLVALIDATE, 0, ACTION_InstallValidate, NULL }, > + { 0, szIsolateComponents, 0, 0, ACTION_IsolateComponents, NULL }, > + { 0, szLaunchConditions, IDS_DESC_LAUNCHCONDITIONS, 0, ACTION_LaunchConditions, NULL }, > + { 0, szMigrateFeatureStates, IDS_DESC_MIGRATEFEATURESTATES, IDS_TEMP_MIGRATEFEATURESTATES, ACTION_MigrateFeatureStates, NULL }, > + { 0, szMoveFiles, IDS_DESC_MOVEFILES, IDS_TEMP_MOVEFILES, ACTION_MoveFiles, NULL }, > + { 1, szMsiPublishAssemblies, IDS_DESC_MSIPUBLISHASSEMBLIES, IDS_TEMP_MSIPUBLISHASSEMBLIES, ACTION_MsiPublishAssemblies, szMsiUnpublishAssemblies }, > + { 1, szMsiUnpublishAssemblies, IDS_DESC_MSIUNPUBLISHASSEMBLIES, IDS_TEMP_MSIUNPUBLISHASSEMBLIES, ACTION_MsiUnpublishAssemblies, szMsiPublishAssemblies }, > + { 0, szPatchFiles, IDS_DESC_PATCHFILES, IDS_TEMP_PATCHFILES, ACTION_PatchFiles, NULL }, If the criterion is making modifications to the system, why are MoveFiles and PatchFiles not deferred?

2 1

Re: [PATCH 4/5] wined3d: Add texture flag for mipmap generation.
by Henri Verbeet 21 Sep '17

21 Sep '17

On 20 September 2017 at 19:53, Matteo Bruni <mbruni(a)codeweavers.com> wrote: > Signed-off-by: Matteo Bruni <mbruni(a)codeweavers.com> > --- > Tests (later on in the patch series) show that calling GenerateMips() > on a texture without the D3D11_RESOURCE_MISC_GENERATE_MIPS flag does > nothing. In theory, the whole check could be done on the d3d11 side > but that turns out to be somewhat awkward, in practice, so I'm going > to add a wined3d flag and pass it around instead. > In principle that's fine, but like this it's unused. (For what it's worth, I don't necessarily think it needs to be used a lot initially. Even if it's only used to print the FIXME or not in wined3d_texture_generate_mipmaps() that would be a start.) There's also the matter of how this relates to WINED3DUSAGE_AUTOGENMIPMAP. This flag is for manual mipmap generation, while WINED3DUSAGE_AUTOGENMIPMAP is for automatic mipmap generation, but it's not pretty that one is a texture flag while the other is a resource usage flag.

1 0

Re: [PATCH 2/3] msi/tests: Add a helper to delete an installed package.
by Marvin 21 Sep '17

21 Sep '17

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=33206 Your paranoid android. === w7u (32 bit install) === The task timed out

2 1

Re: ntdll: Implement set_cpu_context for ARM64 (try 2)
by Martin Storsjö 21 Sep '17

21 Sep '17

On Wed, 20 Sep 2017, André Hentschel wrote: > Signed-off-by: André Hentschel <nerv(a)dawncrow.de> > --- > dlls/ntdll/signal_arm64.c | 41 +++++++++++++++++++++++++++++++++++++---- > 1 file changed, 37 insertions(+), 4 deletions(-) Now that you're implementing this, I think it'd make sense to complete the arm64 CONTEXT struct (compared to the win10 sdk); our version is lacking the float registers (and debug registers) at the end, when compared with the version of the struct in the public SDK. And contrary to ARM, support for the float registers is on by default in all toolchains, so there's no combination where one would be building wine for arm64 where that wouldn't work. Our version also defines the X registers slightly differently, the public SDK uses an union that allows accessing them both via member names and via an array - although it should end up as the same binary representation. I haven't sent any patch for bringing this up to date yet since I wasn't touching this area yet. // Martin

1 0

Re: cmd: Added support for "&&" and "||" condition checking.
by Alexandre Julliard 21 Sep '17

21 Sep '17

Alexander Coffin <alexcoffin1999(a)gmail.com> writes: > @@ -1478,7 +1557,9 @@ void WCMD_execute (const WCHAR *command, const WCHAR *redirects, > switch (i) { > > case WCMD_CALL: > + push_errorlevel_changed(); > WCMD_call (p); > + pop_errorlevel_changed(); This seems to be the only place that does this, so I don't see why you need to manage your own stack instead of saving/restoring things on the process stack. Also returning a success/failure value from the various functions that implement commands would probably be cleaner than adding a global 'changed' flag. -- Alexandre Julliard julliard(a)winehq.org

2 1

How to proceed with shader errors for d3d11 game under wine
by pecisk＠gmail.com 20 Sep '17

20 Sep '17

Hi! I am trying to find better way to collect and report information about game I want to make run under wine "Elite: Dangerous". At the launch of the game game compiles both regular shaders it is using in game and also compute shaders it's using for scenery generation. During that initial compilation game hangs and I see lot of these and similar errors: fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x800002c2. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x00199983. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x800002c2. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x00199983. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x800002c2. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x00199983. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x800002c2. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x00199983. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x800002c2. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x00199983. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x800002c2. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x00199983. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x800002c2. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x00199983. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x800002c2. fixme:d3d_shader:shader_sm4_read_instruction_modifier Unhandled modifier 0x00199983. fixme:d3d_shader:print_glsl_info_log Info log received from GLSL shader #162: fixme:d3d_shader:print_glsl_info_log Compute info fixme:d3d_shader:print_glsl_info_log ------------ fixme:d3d_shader:print_glsl_info_log 0(22) : warning C7050: "R0.zw" might be used before being initialized fixme:d3d_shader:print_glsl_info_log 0(23) : warning C7050: "R1.zw" might be used before being initialized So question is - is there's a way to debug these more in detail? Are any such info useful for people working on d3d11 support? Thank you for any hints, Respectfully, Peteris Krisjanis.

1 0

Re: [v4 2/2] kernel32/tests: pipe: Added test for pipe security attributes
by Jacek Caban 20 Sep '17

20 Sep '17

Hi Jonathan, For a quick test I made modification in your tests, see the attachment. With that modification, tests still pass on Windows but not on Wine (with or without your patches). This shows that there is something wrong with default security descriptor. I don't know what's the exact problem you're trying to fix, but it's likely that fixing default security descriptor is what would fix your problem, I would suggest to concentrate on that first. Thanks, Jacek

1 0

Re: ntdll: Implement set_cpu_context for ARM64 (try 2)
by Alexandre Julliard 20 Sep '17

20 Sep '17

André Hentschel <nerv(a)dawncrow.de> writes: > + "ldr x15, [x0, #0x100]\n\t" > + "mov sp, x15\n\t" > + "ldr x15, [x0, #0x108]\n\t" > + "ldr x0, [x0, #0x8]\n\t" > + "br x15\n\t" This doesn't look very different from the previous try, and it still has the same problem. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: [PATCH 3/5] wined3d: Advertise AUTOGENMIPMAP only for renderable and filterable formats.
by Marvin 20 Sep '17

20 Sep '17

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=33191 Your paranoid android. === w8 (32 bit device) === device.c:6945: Test failed: Got unexpected hr 0. === w864 (32 bit device) === device.c:6945: Test failed: Got unexpected hr 0. === w1064 (32 bit device) === device.c:6945: Test failed: Got unexpected hr 0. === w864 (64 bit device) === device.c:6945: Test failed: Got unexpected hr 0. === w1064 (64 bit device) === device.c:6945: Test failed: Got unexpected hr 0.

1 0

Re: [PATCH 2/5] d3d9/tests: Add a test for nonrenderable format D3DUSAGE_AUTOGENMIPMAP support.
by Marvin 20 Sep '17

20 Sep '17

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at https://testbot.winehq.org/JobDetails.pl?Key=33190 Your paranoid android. === w8 (32 bit device) === device.c:6945: Test failed: Got unexpected hr 0. === w864 (32 bit device) === device.c:6945: Test failed: Got unexpected hr 0. === w1064 (32 bit device) === device.c:6945: Test failed: Got unexpected hr 0. === w864 (64 bit device) === device.c:6945: Test failed: Got unexpected hr 0. === w1064 (64 bit device) === device.c:6945: Test failed: Got unexpected hr 0.

1 0

Re: [PATCH] GdiPlus: Add GdipCreateLineBrushFromRectWithAngle tests and fix compat
by Vincent Povirk 20 Sep '17

20 Sep '17

This causes warnings: ../../../../dlls/gdiplus/tests/brush.c: In function ‘test_createLineBrushFromRectWithAngle’: ../../../../dlls/gdiplus/tests/brush.c:85:5: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement] GpRectF rect2 = { 1, 3, -1, -2 }; ^ ../../../../dlls/gdiplus/tests/brush.c:108:5: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement] GpRectF rect3 = { 1, 3, 0, 1 }; ^ ../../../../dlls/gdiplus/tests/brush.c:112:5: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement] GpRectF rect4 = { 1, 3, 1, 0 }; ^

1 0

Re: [1/3] ole32: Add a trace to IDataObject::GetData.
by Huw Davies 20 Sep '17

20 Sep '17

On Wed, Sep 20, 2017 at 02:54:05PM +0300, Dmitry Timoshkov wrote: > Signed-off-by: Dmitry Timoshkov <dmitry(a)baikal.ru> > --- > dlls/ole32/datacache.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/dlls/ole32/datacache.c b/dlls/ole32/datacache.c > index ecae89d284..4e684b4189 100644 > --- a/dlls/ole32/datacache.c > +++ b/dlls/ole32/datacache.c > @@ -1245,6 +1245,8 @@ static HRESULT WINAPI DataCache_GetData( > DataCache *This = impl_from_IDataObject(iface); > DataCacheEntry *cache_entry; > > + TRACE("%p,%s,%p\n", iface, debugstr_formatetc(pformatetcIn), pmedium); > + > memset(pmedium, 0, sizeof(*pmedium)); > > cache_entry = DataCache_GetEntryForFormatEtc(This, pformatetcIn); I've resent this one and [2/3] with spaces between the commas in the TRACE()s. I hope that's ok with you. Huw.

2 1

Re: [PATCH] include: Remove D3D_DISABLE_9EX checks.
by Henri Verbeet 20 Sep '17

20 Sep '17

On 20 September 2017 at 07:13, Alex Henrie <alexhenrie24(a)gmail.com> wrote: > Fixes https://bugs.winehq.org/show_bug.cgi?id=37160 > > The other option is to add D3D_DISABLE_9EX checks to d3d9.h, but I don't > think it will break anything to declare the new stuff unconditionally. > I'd rather do it right.

1 0

[PATCH 4/4] include: Add dxgi1_6.idl
by Shinchiro Shinchiro 20 Sep '17

20 Sep '17

When I try compiling this idl with widl, I'm getting segfault. I can't figure what's wrong with it

1 1

Re: [PATCH v2 2/2] msi: Store the current script in the package.
by Hans Leidekker 19 Sep '17

19 Sep '17

On Tue, 2017-09-19 at 11:59 -0500, Zebediah Figura wrote: > Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> > --- > dlls/msi/action.c | 25 +++++++++++++++---------- > dlls/msi/custom.c | 34 ++-------------------------------- > dlls/msi/dialog.c | 2 +- > dlls/msi/install.c | 8 ++++---- > dlls/msi/msi.c | 6 +++--- > dlls/msi/msipriv.h | 8 +++----- > dlls/msi/package.c | 1 + > 7 files changed, 29 insertions(+), 55 deletions(-) action.c: In function ‘execute_script’: action.c:1549:16: warning: assignment makes integer from pointer without a cast for (i = package->script_actions_count; i > 0; i--) ^

1 0

Re: [PATCH v2 1/2] msi: Remove a superfluous substructure.
by Hans Leidekker 19 Sep '17

19 Sep '17

On Tue, 2017-09-19 at 11:58 -0500, Zebediah Figura wrote: > v2: don't use lists > Signed-off-by: Zebediah Figura <z.figura12(a)gmail.com> > --- > dlls/msi/action.c | 45 ++++++++++++++++++++------------------------- > dlls/msi/custom.c | 39 ++++++++++++++++----------------------- > dlls/msi/msipriv.h | 36 +++++++++++++++--------------------- > dlls/msi/package.c | 31 ++++++++++++------------------- > 4 files changed, 63 insertions(+), 88 deletions(-) This gives a warning: package.c: In function ‘msi_alloc_package’: package.c:999:9: warning: unused variable ‘i’ [-Wunused-variable] int i; ^

1 0

Re: [PATCH 3/3] msi: Remove properties from deferred custom actions.
by Hans Leidekker 19 Sep '17

19 Sep '17

On Tue, 2017-09-19 at 00:23 -0500, Zebediah Figura wrote: > This effectively reverts 3c444ea350e0688247bd2056b3861dac4af08fbc, > which should not be necessary, and in fact does not affect the > linked application, MSN Messenger. This needs a test.

2 1

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-devel