May 2017 - Wine-commits - list.winehq.org

Julian Rüger : German translation for release 2.7
by Jeremy Newman 03 May '17

03 May '17

Module: website Branch: master Commit: eb22809be5537eee2a690c56f4be1f11f537c615 URL: http://source.winehq.org/git/website.git/?a=commit;h=eb22809be5537eee2a690c… Author: Julian Rüger <jr98(a)gmx.net> Date: Wed May 3 13:49:24 2017 +0200 German translation for release 2.7 Signed-off-by: Julian Rüger <jr98(a)gmx.net> Signed-off-by: Jeremy Newman <jnewman(a)codeweavers.com> --- news/de/2017042801.xml | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/news/de/2017042801.xml b/news/de/2017042801.xml new file mode 100644 index 0000000..72b4040 --- /dev/null +++ b/news/de/2017042801.xml @@ -0,0 +1,18 @@ +<news> +<date>28. April 2017</date> +<title>Wine 2.7 freigegeben</title> +<body> + Die Entwicklungsversion 2.7 von Wine ist jetzt verfügbar. + <a href="{$root}/announce/2.7">Neuerungen (en)</a> in dieser Version: +<ul> + <li>Unterstützung von TCP- und UDP-Verbindungen in WebServices.</li> + <li>Diverse Shader-Verbesserungen für Direct3D 11.</li> + <li>Verbesserte Unterstützung von hohen DPI-Einstellungen.</li> + <li>Partielle Reimplementierung der GLU-Library.</li> + <li>Unterstützung von aktuellen Versionen von OSMesa.</li> + <li>Fenstermanagement-Verbesserungen unter macOS.</li> + <li>Diverse Fehlerkorrekturen.</li> +</ul> +Der Quelltext ist ab sofort <a href="//dl.winehq.org/wine/source/2.x/wine-2.7.tar.xz">verfügbar</a>. +Binärpakete werden gerade erstellt und stehen bald auf den jeweiligen <a href="{$root}/download">Downloadseiten</a> zur Verfügung. +</body></news>

1 0

Michael Stefaniuc : dmusic: Avoid an intermediate copy in PackStructured() .
by Alexandre Julliard 02 May '17

02 May '17

Module: wine Branch: master Commit: 950b8a9699fc640b0b58126ce38d25ddaf91dac9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=950b8a9699fc640b0b58126ce… Author: Michael Stefaniuc <mstefani(a)winehq.org> Date: Mon May 1 23:58:50 2017 +0200 dmusic: Avoid an intermediate copy in PackStructured(). Signed-off-by: Michael Stefaniuc <mstefani(a)winehq.org> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dmusic/buffer.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/dlls/dmusic/buffer.c b/dlls/dmusic/buffer.c index dbc8058..9b0ca84 100644 --- a/dlls/dmusic/buffer.c +++ b/dlls/dmusic/buffer.c @@ -101,7 +101,7 @@ static HRESULT WINAPI IDirectMusicBufferImpl_PackStructured(LPDIRECTMUSICBUFFER { IDirectMusicBufferImpl *This = impl_from_IDirectMusicBuffer(iface); DWORD new_write_pos = This->write_pos + DMUS_EVENT_SIZE(sizeof(channel_message)); - DMUS_EVENTHEADER header; + DMUS_EVENTHEADER *header; TRACE("(%p)->(0x%s, %u, 0x%x)\n", iface, wine_dbgstr_longlong(ref_time), channel_group, channel_message); @@ -119,13 +119,13 @@ static HRESULT WINAPI IDirectMusicBufferImpl_PackStructured(LPDIRECTMUSICBUFFER if (!This->write_pos) This->start_time = ref_time; - header.cbEvent = 3; /* Midi message takes 4 bytes space but only 3 are relevant */ - header.dwChannelGroup = channel_group; - header.rtDelta = ref_time - This->start_time; - header.dwFlags = DMUS_EVENT_STRUCTURED; + header = (DMUS_EVENTHEADER*)&This->data[This->write_pos]; + header->cbEvent = 3; /* Midi message takes 4 bytes space but only 3 are relevant */ + header->dwChannelGroup = channel_group; + header->rtDelta = ref_time - This->start_time; + header->dwFlags = DMUS_EVENT_STRUCTURED; - memcpy(This->data + This->write_pos, &header, sizeof(header)); - *(DWORD*)(This->data + This->write_pos + sizeof(header)) = channel_message; + *(DWORD*)&header[1] = channel_message; This->write_pos = new_write_pos; return S_OK;

1 0

Andrew Wesie : setupapi: Fix SetupDiGetDeviceRegistryProperty if property does not exist.
by Alexandre Julliard 02 May '17

02 May '17

Module: wine Branch: master Commit: e4645d60d72a9fbfbd745e275d4f90110bdceb4d URL: http://source.winehq.org/git/wine.git/?a=commit;h=e4645d60d72a9fbfbd745e275… Author: Andrew Wesie <awesie(a)gmail.com> Date: Mon May 1 22:14:57 2017 -0500 setupapi: Fix SetupDiGetDeviceRegistryProperty if property does not exist. Signed-off-by: Andrew Wesie <awesie(a)gmail.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/setupapi/devinst.c | 8 ++++++-- dlls/setupapi/tests/devinst.c | 8 ++++++++ 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/dlls/setupapi/devinst.c b/dlls/setupapi/devinst.c index ffb5fad..4a852da 100644 --- a/dlls/setupapi/devinst.c +++ b/dlls/setupapi/devinst.c @@ -3125,7 +3125,9 @@ BOOL WINAPI SetupDiGetDeviceRegistryPropertyA( LONG l = RegQueryValueExA(devInfo->key, PropertyMap[Property].nameA, NULL, PropertyRegDataType, PropertyBuffer, &size); - if (l == ERROR_MORE_DATA || !PropertyBufferSize) + if (l == ERROR_FILE_NOT_FOUND) + SetLastError(ERROR_INVALID_DATA); + else if (l == ERROR_MORE_DATA || !PropertyBufferSize) SetLastError(ERROR_INSUFFICIENT_BUFFER); else if (!l) ret = TRUE; @@ -3186,7 +3188,9 @@ BOOL WINAPI SetupDiGetDeviceRegistryPropertyW( LONG l = RegQueryValueExW(devInfo->key, PropertyMap[Property].nameW, NULL, PropertyRegDataType, PropertyBuffer, &size); - if (l == ERROR_MORE_DATA || !PropertyBufferSize) + if (l == ERROR_FILE_NOT_FOUND) + SetLastError(ERROR_INVALID_DATA); + else if (l == ERROR_MORE_DATA || !PropertyBufferSize) SetLastError(ERROR_INSUFFICIENT_BUFFER); else if (!l) ret = TRUE; diff --git a/dlls/setupapi/tests/devinst.c b/dlls/setupapi/tests/devinst.c index 6ca0e92..4df00b1 100644 --- a/dlls/setupapi/tests/devinst.c +++ b/dlls/setupapi/tests/devinst.c @@ -1053,6 +1053,10 @@ static void testDeviceRegistryPropertyA(void) todo_wine ok(!ret && GetLastError() == ERROR_INVALID_DATA, "Expected ERROR_INVALID_DATA, got %08x\n", GetLastError()); + ret = pSetupDiGetDeviceRegistryPropertyA(set, &devInfo, SPDRP_HARDWAREID, + NULL, NULL, 0, &size); + ok(!ret && GetLastError() == ERROR_INVALID_DATA, + "Expected ERROR_INVALID_DATA, got %08x\n", GetLastError()); pSetupDiDestroyDeviceInfoList(set); res = RegOpenKeyA(HKEY_LOCAL_MACHINE, bogus, &key); @@ -1158,6 +1162,10 @@ static void testDeviceRegistryPropertyW(void) todo_wine ok(!ret && GetLastError() == ERROR_INVALID_DATA, "Expected ERROR_INVALID_DATA, got %08x\n", GetLastError()); + ret = pSetupDiGetDeviceRegistryPropertyW(set, &devInfo, SPDRP_HARDWAREID, + NULL, NULL, 0, &size); + ok(!ret && GetLastError() == ERROR_INVALID_DATA, + "Expected ERROR_INVALID_DATA, got %08x\n", GetLastError()); pSetupDiDestroyDeviceInfoList(set); res = RegOpenKeyW(HKEY_LOCAL_MACHINE, bogus, &key);

1 0

Henri Verbeet : ddraw/tests: Port test_texturemapblend() to ddraw4.c as well.
by Alexandre Julliard 02 May '17

02 May '17

Module: wine Branch: master Commit: 7f826a7131dbdf6b39c54d8eab9c8c759f359382 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7f826a7131dbdf6b39c54d8ea… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue May 2 15:31:03 2017 +0200 ddraw/tests: Port test_texturemapblend() to ddraw4.c as well. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ddraw/tests/ddraw4.c | 348 ++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 348 insertions(+) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=7f826a7131dbdf6b39c54…

1 0

Henri Verbeet : ddraw: Validate that non-fill blits have a source surface in ddraw_surface7_Blt( ).
by Alexandre Julliard 02 May '17

02 May '17

Module: wine Branch: master Commit: fb4740bbb39ddfd017387b18c40c8582166f746c URL: http://source.winehq.org/git/wine.git/?a=commit;h=fb4740bbb39ddfd017387b18c… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue May 2 15:31:04 2017 +0200 ddraw: Validate that non-fill blits have a source surface in ddraw_surface7_Blt(). Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ddraw/surface.c | 6 ++++++ dlls/ddraw/tests/ddraw1.c | 7 +++++++ dlls/ddraw/tests/ddraw2.c | 7 +++++++ dlls/ddraw/tests/ddraw4.c | 15 +++++++++++---- dlls/ddraw/tests/ddraw7.c | 15 +++++++++++---- 5 files changed, 42 insertions(+), 8 deletions(-) diff --git a/dlls/ddraw/surface.c b/dlls/ddraw/surface.c index a780f1a..394e915 100644 --- a/dlls/ddraw/surface.c +++ b/dlls/ddraw/surface.c @@ -1710,6 +1710,12 @@ static HRESULT WINAPI DECLSPEC_HOTPATCH ddraw_surface7_Blt(IDirectDrawSurface7 * } } + if (!(flags & (DDBLT_COLORFILL | DDBLT_DEPTHFILL)) && !src_impl) + { + WARN("No source surface.\n"); + return DDERR_INVALIDPARAMS; + } + if (flags & DDBLT_KEYSRC && (!src_impl || !(src_impl->surface_desc.dwFlags & DDSD_CKSRCBLT))) { WARN("DDBLT_KEYSRC blit without color key in surface, returning DDERR_INVALIDPARAMS\n"); diff --git a/dlls/ddraw/tests/ddraw1.c b/dlls/ddraw/tests/ddraw1.c index ac4942d..c052c03 100644 --- a/dlls/ddraw/tests/ddraw1.c +++ b/dlls/ddraw/tests/ddraw1.c @@ -8863,6 +8863,13 @@ static void test_blt(void) hr = IDirectDrawSurface_Blt(surface, &test_data[i].dst_rect, rt, &test_data[i].src_rect, DDBLT_WAIT, NULL); ok(hr == test_data[i].hr, "Test %u: Got unexpected hr %#x, expected %#x.\n", i, hr, test_data[i].hr); + + hr = IDirectDrawSurface_Blt(surface, &test_data[i].dst_rect, + NULL, &test_data[i].src_rect, DDBLT_WAIT, NULL); + ok(hr == DDERR_INVALIDPARAMS, "Test %u: Got unexpected hr %#x.\n", i, hr); + + hr = IDirectDrawSurface_Blt(surface, &test_data[i].dst_rect, NULL, NULL, DDBLT_WAIT, NULL); + ok(hr == DDERR_INVALIDPARAMS, "Test %u: Got unexpected hr %#x.\n", i, hr); } IDirectDrawSurface_Release(surface); diff --git a/dlls/ddraw/tests/ddraw2.c b/dlls/ddraw/tests/ddraw2.c index 67a2b06..f81a296 100644 --- a/dlls/ddraw/tests/ddraw2.c +++ b/dlls/ddraw/tests/ddraw2.c @@ -9951,6 +9951,13 @@ static void test_blt(void) hr = IDirectDrawSurface_Blt(surface, &test_data[i].dst_rect, rt, &test_data[i].src_rect, DDBLT_WAIT, NULL); ok(hr == test_data[i].hr, "Test %u: Got unexpected hr %#x, expected %#x.\n", i, hr, test_data[i].hr); + + hr = IDirectDrawSurface_Blt(surface, &test_data[i].dst_rect, + NULL, &test_data[i].src_rect, DDBLT_WAIT, NULL); + ok(hr == DDERR_INVALIDPARAMS, "Test %u: Got unexpected hr %#x.\n", i, hr); + + hr = IDirectDrawSurface_Blt(surface, &test_data[i].dst_rect, NULL, NULL, DDBLT_WAIT, NULL); + ok(hr == DDERR_INVALIDPARAMS, "Test %u: Got unexpected hr %#x.\n", i, hr); } IDirectDrawSurface_Release(surface); diff --git a/dlls/ddraw/tests/ddraw4.c b/dlls/ddraw/tests/ddraw4.c index 763151a..f25b327 100644 --- a/dlls/ddraw/tests/ddraw4.c +++ b/dlls/ddraw/tests/ddraw4.c @@ -11395,21 +11395,28 @@ static void test_blt(void) hr = IDirectDraw4_CreateSurface(ddraw, &surface_desc, &surface, NULL); ok(SUCCEEDED(hr), "Failed to create surface, hr %#x.\n", hr); - hr = IDirectDrawSurface_Blt(surface, NULL, surface, NULL, 0, NULL); + hr = IDirectDrawSurface4_Blt(surface, NULL, surface, NULL, 0, NULL); ok(SUCCEEDED(hr), "Failed to blit, hr %#x.\n", hr); - hr = IDirectDrawSurface_Blt(surface, NULL, rt, NULL, 0, NULL); + hr = IDirectDrawSurface4_Blt(surface, NULL, rt, NULL, 0, NULL); ok(SUCCEEDED(hr), "Failed to blit, hr %#x.\n", hr); for (i = 0; i < sizeof(test_data) / sizeof(*test_data); ++i) { - hr = IDirectDrawSurface_Blt(surface, &test_data[i].dst_rect, + hr = IDirectDrawSurface4_Blt(surface, &test_data[i].dst_rect, surface, &test_data[i].src_rect, DDBLT_WAIT, NULL); ok(hr == test_data[i].hr, "Test %u: Got unexpected hr %#x, expected %#x.\n", i, hr, test_data[i].hr); - hr = IDirectDrawSurface_Blt(surface, &test_data[i].dst_rect, + hr = IDirectDrawSurface4_Blt(surface, &test_data[i].dst_rect, rt, &test_data[i].src_rect, DDBLT_WAIT, NULL); ok(hr == test_data[i].hr, "Test %u: Got unexpected hr %#x, expected %#x.\n", i, hr, test_data[i].hr); + + hr = IDirectDrawSurface4_Blt(surface, &test_data[i].dst_rect, + NULL, &test_data[i].src_rect, DDBLT_WAIT, NULL); + ok(hr == DDERR_INVALIDPARAMS, "Test %u: Got unexpected hr %#x.\n", i, hr); + + hr = IDirectDrawSurface4_Blt(surface, &test_data[i].dst_rect, NULL, NULL, DDBLT_WAIT, NULL); + ok(hr == DDERR_INVALIDPARAMS, "Test %u: Got unexpected hr %#x.\n", i, hr); } IDirectDrawSurface4_Release(surface); diff --git a/dlls/ddraw/tests/ddraw7.c b/dlls/ddraw/tests/ddraw7.c index 79ce80f..354e872 100644 --- a/dlls/ddraw/tests/ddraw7.c +++ b/dlls/ddraw/tests/ddraw7.c @@ -11374,21 +11374,28 @@ static void test_blt(void) hr = IDirectDraw7_CreateSurface(ddraw, &surface_desc, &surface, NULL); ok(SUCCEEDED(hr), "Failed to create surface, hr %#x.\n", hr); - hr = IDirectDrawSurface_Blt(surface, NULL, surface, NULL, 0, NULL); + hr = IDirectDrawSurface7_Blt(surface, NULL, surface, NULL, 0, NULL); ok(SUCCEEDED(hr), "Failed to blit, hr %#x.\n", hr); - hr = IDirectDrawSurface_Blt(surface, NULL, rt, NULL, 0, NULL); + hr = IDirectDrawSurface7_Blt(surface, NULL, rt, NULL, 0, NULL); ok(SUCCEEDED(hr), "Failed to blit, hr %#x.\n", hr); for (i = 0; i < sizeof(test_data) / sizeof(*test_data); ++i) { - hr = IDirectDrawSurface_Blt(surface, &test_data[i].dst_rect, + hr = IDirectDrawSurface7_Blt(surface, &test_data[i].dst_rect, surface, &test_data[i].src_rect, DDBLT_WAIT, NULL); ok(hr == test_data[i].hr, "Test %u: Got unexpected hr %#x, expected %#x.\n", i, hr, test_data[i].hr); - hr = IDirectDrawSurface_Blt(surface, &test_data[i].dst_rect, + hr = IDirectDrawSurface7_Blt(surface, &test_data[i].dst_rect, rt, &test_data[i].src_rect, DDBLT_WAIT, NULL); ok(hr == test_data[i].hr, "Test %u: Got unexpected hr %#x, expected %#x.\n", i, hr, test_data[i].hr); + + hr = IDirectDrawSurface7_Blt(surface, &test_data[i].dst_rect, + NULL, &test_data[i].src_rect, DDBLT_WAIT, NULL); + ok(hr == DDERR_INVALIDPARAMS, "Test %u: Got unexpected hr %#x.\n", i, hr); + + hr = IDirectDrawSurface7_Blt(surface, &test_data[i].dst_rect, NULL, NULL, DDBLT_WAIT, NULL); + ok(hr == DDERR_INVALIDPARAMS, "Test %u: Got unexpected hr %#x.\n", i, hr); } IDirectDrawSurface7_Release(surface);

1 0

Henri Verbeet : wined3d: Print a FIXME if the view format doesn' t match the resource format in surface_cpu_blt_colour_fill().
by Alexandre Julliard 02 May '17

02 May '17

Module: wine Branch: master Commit: aa199aca5c8dbe17f0752b3c24a32f1572acab58 URL: http://source.winehq.org/git/wine.git/?a=commit;h=aa199aca5c8dbe17f0752b3c2… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue May 2 11:35:28 2017 +0200 wined3d: Print a FIXME if the view format doesn't match the resource format in surface_cpu_blt_colour_fill(). Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/surface.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index a02b6f2..864a20c 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -3359,6 +3359,10 @@ static void surface_cpu_blt_colour_fill(struct wined3d_rendertarget_view *view, return; } + if (view->format->id != view->resource->format->id) + FIXME("View format %s doesn't match resource format %s.\n", + debug_d3dformat(view->format->id), debug_d3dformat(view->resource->format->id)); + if (view->resource->type == WINED3D_RTYPE_BUFFER) { FIXME("Not implemented for buffers.\n");

1 0

Henri Verbeet : wined3d: Use the CPU blitter for clearing depth/ stencil textures that are current in the map binding.
by Alexandre Julliard 02 May '17

02 May '17

Module: wine Branch: master Commit: 2f0b9452358125063f03cadde77374e030136daf URL: http://source.winehq.org/git/wine.git/?a=commit;h=2f0b9452358125063f03cadde… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue May 2 11:35:29 2017 +0200 wined3d: Use the CPU blitter for clearing depth/stencil textures that are current in the map binding. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/surface.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 864a20c..e86119d 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -2596,7 +2596,9 @@ static void ffp_blitter_clear(struct wined3d_blitter *blitter, struct wined3d_de if (flags & (WINED3DCLEAR_ZBUFFER | WINED3DCLEAR_STENCIL)) { - if (fb->depth_stencil && fb->depth_stencil->resource->pool == WINED3D_POOL_SYSTEM_MEM) + view = fb->depth_stencil; + if (view && (view->resource->pool == WINED3D_POOL_SYSTEM_MEM + || ffp_blitter_use_cpu_clear(view))) goto next; }

1 0

Henri Verbeet : wined3d: Allow the command stream thread to run asynchronously.
by Alexandre Julliard 02 May '17

02 May '17

Module: wine Branch: master Commit: 73170152dc66b5cee437e687d04d6cc2606b6797 URL: http://source.winehq.org/git/wine.git/?a=commit;h=73170152dc66b5cee437e687d… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue May 2 11:35:30 2017 +0200 wined3d: Allow the command stream thread to run asynchronously. For a modest performance improvement in CPU-limited scenarios, especially ones where a significant amount of time is spent in the graphics driver. There's certainly still more room for improvement, resource maps in particular will currently cause stalls. Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wined3d/cs.c | 46 +++++++++++++++++++++++++++++++++++++++--- dlls/wined3d/device.c | 6 ++++++ dlls/wined3d/swapchain.c | 10 +++++++++ dlls/wined3d/texture.c | 11 ++++++++++ dlls/wined3d/wined3d_private.h | 2 ++ 5 files changed, 72 insertions(+), 3 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=73170152dc66b5cee437e…

1 0

Huw Davies : explorer: Set the rebar band height to match the combobox.
by Alexandre Julliard 02 May '17

02 May '17

Module: wine Branch: master Commit: db52c6d074eded1d5312f25257a5ca89823e3a72 URL: http://source.winehq.org/git/wine.git/?a=commit;h=db52c6d074eded1d5312f2525… Author: Huw Davies <huw(a)codeweavers.com> Date: Tue May 2 11:03:18 2017 +0100 explorer: Set the rebar band height to match the combobox. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/explorer/explorer.c | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/programs/explorer/explorer.c b/programs/explorer/explorer.c index 43466dc..cc3244d 100644 --- a/programs/explorer/explorer.c +++ b/programs/explorer/explorer.c @@ -301,7 +301,7 @@ static IExplorerBrowserEvents *make_explorer_events(explorer_info *info) static void make_explorer_window(IShellFolder* startFolder) { - RECT explorerRect; + RECT rect; HWND rebar,nav_toolbar; FOLDERSETTINGS fs; IExplorerBrowserEvents *events; @@ -340,12 +340,9 @@ static void make_explorer_window(IShellFolder* startFolder) fs.ViewMode = FVM_DETAILS; fs.fFlags = FWF_AUTOARRANGE; - explorerRect.left = 0; - explorerRect.top = 0; - explorerRect.right = DEFAULT_WIDTH; - explorerRect.bottom = DEFAULT_HEIGHT; - IExplorerBrowser_Initialize(info->browser,info->main_window,&explorerRect,&fs); + SetRect(&rect, 0, 0, DEFAULT_WIDTH, DEFAULT_HEIGHT); + IExplorerBrowser_Initialize(info->browser,info->main_window,&rect,&fs); IExplorerBrowser_SetOptions(info->browser,EBO_SHOWFRAMES); SetWindowLongPtrW(info->main_window,EXPLORER_INFO_INDEX,(LONG_PTR)info); @@ -392,9 +389,10 @@ static void make_explorer_window(IShellFolder* startFolder) WS_CHILD | WS_VISIBLE | CBS_DROPDOWN, 0,0,DEFAULT_WIDTH,PATHBOX_HEIGHT,rebar,NULL, explorer_hInstance,NULL); - band_info.cyChild=PATHBOX_HEIGHT; + GetWindowRect(info->path_box, &rect); + band_info.cyChild = rect.bottom - rect.top; band_info.cx=0; - band_info.cyMinChild=PATHBOX_HEIGHT; + band_info.cyMinChild = rect.bottom - rect.top; band_info.cxMinChild=0; band_info.fMask|=RBBIM_TEXT; band_info.lpText=pathbox_label;

1 0

Huw Davies : wordpad: Scale the comboboxes with font size and correctly set the rebar band height.
by Alexandre Julliard 02 May '17

02 May '17

Module: wine Branch: master Commit: 8d22b60c8006564b154ccd9689b5f3948a6a127d URL: http://source.winehq.org/git/wine.git/?a=commit;h=8d22b60c8006564b154ccd968… Author: Huw Davies <huw(a)codeweavers.com> Date: Tue May 2 11:03:19 2017 +0100 wordpad: Scale the comboboxes with font size and correctly set the rebar band height. Signed-off-by: Huw Davies <huw(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- programs/wordpad/wordpad.c | 31 ++++++++++++++++++++++++------- 1 file changed, 24 insertions(+), 7 deletions(-) diff --git a/programs/wordpad/wordpad.c b/programs/wordpad/wordpad.c index 01a9da2..f91274d 100644 --- a/programs/wordpad/wordpad.c +++ b/programs/wordpad/wordpad.c @@ -1813,8 +1813,15 @@ static LRESULT OnCreate( HWND hWnd ) int nStdBitmaps = 0; REBARINFO rbi; REBARBANDINFOW rbb; + RECT rect; + HFONT font; + HDC hdc; + SIZE name_sz, size_sz; + int height; static const WCHAR wszRichEditDll[] = {'R','I','C','H','E','D','2','0','.','D','L','L','\0'}; static const WCHAR wszRichEditText[] = {'R','i','c','h','E','d','i','t',' ','t','e','x','t','\0'}; + static const WCHAR font_text[] = {'T','i','m','e','s',' ','N','e','w',' ','R','o','m','a','n',0}; /* a long font name */ + static const WCHAR size_text[] = {' ','0','0',0}; /* enough for two digits */ CreateStatusWindowW(CCS_NODIVIDER|WS_CHILD|WS_VISIBLE, wszRichEditText, hWnd, IDC_STATUSBAR); @@ -1856,6 +1863,13 @@ static LRESULT OnCreate( HWND hWnd ) AddButton(hToolBarWnd, 0, ID_DATETIME); SendMessageW(hToolBarWnd, TB_AUTOSIZE, 0, 0); + height = HIWORD(SendMessageW(hToolBarWnd, TB_GETBUTTONSIZE, 0, 0)); + + hFontListWnd = CreateWindowExW(0, WC_COMBOBOXEXW, NULL, + WS_BORDER | WS_VISIBLE | WS_CHILD | CBS_DROPDOWN | CBS_SORT, + 0, 0, 200, 150, hReBarWnd, (HMENU)IDC_FONTLIST, hInstance, NULL); + GetWindowRect(hFontListWnd, &rect); + height = max(height, rect.bottom - rect.top); rbb.cbSize = REBARBANDINFOW_V6_SIZE; rbb.fMask = RBBIM_SIZE | RBBIM_CHILDSIZE | RBBIM_CHILD | RBBIM_STYLE | RBBIM_ID; @@ -1863,17 +1877,20 @@ static LRESULT OnCreate( HWND hWnd ) rbb.cx = 0; rbb.hwndChild = hToolBarWnd; rbb.cxMinChild = 0; - rbb.cyChild = rbb.cyMinChild = HIWORD(SendMessageW(hToolBarWnd, TB_GETBUTTONSIZE, 0, 0)); + rbb.cyChild = rbb.cyMinChild = height; rbb.wID = BANDID_TOOLBAR; SendMessageW(hReBarWnd, RB_INSERTBANDW, -1, (LPARAM)&rbb); - hFontListWnd = CreateWindowExW(0, WC_COMBOBOXEXW, NULL, - WS_BORDER | WS_VISIBLE | WS_CHILD | CBS_DROPDOWN | CBS_SORT, - 0, 0, 200, 150, hReBarWnd, (HMENU)IDC_FONTLIST, hInstance, NULL); - + font = (HFONT)SendMessageW(hFontListWnd, WM_GETFONT, 0, 0); + hdc = GetDC(hFontListWnd); + font = SelectObject(hdc, font); + GetTextExtentPointW(hdc, font_text, sizeof(font_text) / sizeof(font_text[0]) - 1, &name_sz); + GetTextExtentPointW(hdc, size_text, sizeof(size_text) / sizeof(size_text[0]) - 1, &size_sz); + font = SelectObject(hdc, font); + ReleaseDC(hFontListWnd, hdc); rbb.hwndChild = hFontListWnd; - rbb.cx = 200; + rbb.cx = MulDiv(name_sz.cx, 3, 2) + height; /* height is space for the dropdown arrow */ rbb.wID = BANDID_FONTLIST; SendMessageW(hReBarWnd, RB_INSERTBANDW, -1, (LPARAM)&rbb); @@ -1883,7 +1900,7 @@ static LRESULT OnCreate( HWND hWnd ) 0, 0, 50, 150, hReBarWnd, (HMENU)IDC_SIZELIST, hInstance, NULL); rbb.hwndChild = hSizeListWnd; - rbb.cx = 50; + rbb.cx = MulDiv(size_sz.cx, 3, 2) + height; /* height is space for the dropdown arrow */ rbb.fStyle ^= RBBS_BREAK; rbb.wID = BANDID_SIZELIST;

1 0