- wine-gitlab - winehq.org

Re: [PATCH v2 0/1] MR1450: xmllite/reader: Use consistent formatting for traces. - approved
by Alexandre Julliard (＠julliard) 21 Nov '22

21 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1450

1 0

Re: [PATCH v2 0/1] MR1452: win32u: Fix a memory leak. - approved
by Alexandre Julliard (＠julliard) 21 Nov '22

21 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1452

1 0

Re: [PATCH v3 0/1] MR1458: appwiz: Use standard C functions for memory allocation. - approved
by Alexandre Julliard (＠julliard) 21 Nov '22

21 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1458

1 0

Re: [PATCH v2 0/1] MR1463: oledb32: Handle memory allocation failure in rowpos_cp_Advise. - approved
by Alexandre Julliard (＠julliard) 21 Nov '22

21 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1463

1 0

Re: [PATCH v2 0/1] MR1457: adsldp: Use standard C functions for memory allocation in adsldp.c. - approved
by Alexandre Julliard (＠julliard) 21 Nov '22

21 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1457

1 0

Re: [PATCH v3 0/5] MR1472: opengl32: Complete wow64 thunks implementation. - approved
by Alexandre Julliard (＠julliard) 21 Nov '22

21 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1472

1 0

Re: [PATCH v4 0/1] MR1449: msado15: Avoid uninitialized variable warning in fields_get_Item - approved
by Alexandre Julliard (＠julliard) 21 Nov '22

21 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1449

1 0

Re: [PATCH v8 0/3] MR1343: windows.media: Add WinRT Closed Captioning interface. - approved
by Alexandre Julliard (＠julliard) 21 Nov '22

21 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1343

1 0

Re: [PATCH v2 0/1] MR1416: comctl32/tests: Fix monthcal selection tests. - approved
by Alexandre Julliard (＠julliard) 21 Nov '22

21 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1416

1 0

Re: [PATCH v2 0/1] MR1471: dwrite: Avoid memcpy source overflow in GetFontAxisValues. - approved
by Alexandre Julliard (＠julliard) 21 Nov '22

21 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1471

1 0

Re: [PATCH v2 0/1] MR1465: secur32: Use the wcsdup function instead of reimplementing it. - approved
by Alexandre Julliard (＠julliard) 21 Nov '22

21 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1465

1 0

Re: [PATCH v2 0/1] MR1468: winhttp: Use the wcsdup function instead of reimplementing it. - approved
by Alexandre Julliard (＠julliard) 21 Nov '22

21 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1468

1 0

Re: [PATCH v2 0/1] MR1469: wldap32: Use the wcsdup function instead of reimplementing it. - approved
by Alexandre Julliard (＠julliard) 21 Nov '22

21 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1469

1 0

[PATCH v2 0/3] MR1046: Draft: MfMediaEngine invalid format handling.
by Bernhard Kölbl (＠besentv) 21 Nov '22

21 Nov '22

-- v2: mfmediaengine: Add a fallback when the set output format cannot be used to init samplegrabber. mfmediaengine: Only accept output formats which are render target compatible. mfmediaengine/tests: Test possible engine and output texture formats. https://gitlab.winehq.org/wine/wine/-/merge_requests/1046

3 4

Re: [PATCH v3 0/2] MR49: Always declare UAV images as typed if atomic ops occur. - approved
by Alexandre Julliard (＠julliard) 21 Nov '22

21 Nov '22

This merge request was approved by Alexandre Julliard. -- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/49

1 0

[PATCH v3 0/3] MR1448: hid: parser and HidP_SetUsageValue fix
by Matthew Tran (＠0e4ef622) 21 Nov '22

21 Nov '22

This fixes two things: * HID report descriptor parser enum for local items. Close inspection of page 40 of [Device Class Definition for HID 1.11](https://www.usb.org/document-library/device-class-definition-hid-111) reveals a jump between Designator Maximum (0101) and String Index (0111). This caused a controller of mine to not get recognized. * HidP_SetUsageValue for items that define multiple controls (with Report Count > 1). Attempting to set one value would result in HIDP_STATUS_BUFFER_TOO_SMALL. -- v3: hid: Add tests for fixed HidP_SetUsageValue hid: Fix HidP_SetUsageValue for usage ranges hidparse.sys: Fix incorrect enum https://gitlab.winehq.org/wine/wine/-/merge_requests/1448

4 10

[PATCH v2 0/12] MR1470: kernel32: Add (Global|Local)ReAlloc tests and fix it.
by Rémi Bernon (＠rbernon) 21 Nov '22

21 Nov '22

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53474 -- v2: kernelbase: Only discard HLOCAL with LMEM_MOVEABLE and if not locked. kernel32: Return success from GlobalReAlloc with locked HGLOBAL pointer. kernelbase: Avoid moving locked HLOCAL pointer without LMEM_MOVEABLE. kernelbase: Handle LMEM_DISCARDABLE flag specifically in LocalReAlloc. kernelbase: Handle LMEM_MODIFY flag first in LocalReAlloc. kernelbase: Don't try in-place HeapReAlloc in LocalReAlloc. user32: Call GlobalReAlloc with GMEM_MOVEABLE flag. winemac.drv: Call GlobalReAlloc with GMEM_MOVEABLE flag. riched20: Call GlobalReAlloc with GMEM_MOVEABLE flag. ole32: Call GlobalReAlloc with GMEM_MOVEABLE flag. comdlg32: Call GlobalReAlloc with GMEM_MOVEABLE flag. kernel32/tests: Add more (Global|Local)ReAlloc tests. https://gitlab.winehq.org/wine/wine/-/merge_requests/1470

3 15

Re: [PATCH v12 0/9] MR1297: gdiplus: Add CustomLineCap implementation (including displaying).
by Esme Povirk (＠madewokherd) 21 Nov '22

21 Nov '22

It occurred to me that you might not be aware that you can forward-declare static functions, so you don't really need to move them. Just put a declaration like this: ``` static void widen_open_figure(const GpPointF *points, int start, int end, GpPen *pen, REAL pen_width, GpLineCap start_cap, GpLineCap end_cap, path_list_node_t **last_point); ``` Further up in the file. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1297#note_16784

1 0

Re: [PATCH v12 0/9] MR1297: gdiplus: Add CustomLineCap implementation (including displaying).
by Esme Povirk (＠madewokherd) 21 Nov '22

21 Nov '22

On Sat Nov 19 00:10:03 2022 +0000, Esme Povirk wrote: > I suspect there are enough slow things we already have to do that it > won't really matter, but if we need to we can optimize it later. Resolving, this is fine for now and can be easily improved in the future. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1297#note_16782

1 0

Re: [PATCH v12 0/9] MR1297: gdiplus: Add CustomLineCap implementation (including displaying).
by Esme Povirk (＠madewokherd) 21 Nov '22

21 Nov '22

On Fri Nov 18 20:11:22 2022 +0000, Esme Povirk wrote: > I'm not sure. Maybe widen_open_figure needs to be aware of the inset and > adjust the points it passes to width_joint. Or maybe the path needs to > be adjusted after GdipFlattenPath. > I think it depends on what happens when the path ends with a segment > that's shorter than the inset. Resolving this thread because this issue doesn't need to be taken care of for the MR to be accepted. It can be fixed later. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1297#note_16780

1 0

Re: [PATCH v12 0/9] MR1297: gdiplus: Add CustomLineCap implementation (including displaying).
by Esme Povirk (＠madewokherd) 21 Nov '22

21 Nov '22

On Wed Nov 9 18:25:51 2022 +0000, Bartosz Kosiorek wrote: > changed this line in [version 5 of the diff](/wine/wine/-/merge_requests/1297/diffs?diff_id=17873&start_sha=ec560a173011779aa36bdb3d54e0bd805a70a729#708701348c59d29ebde2b9d5cd4724966113ebf7_308_308) I'm going to resolve this as it's only an explanation of the change and not a problem with it. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1297#note_16779

1 0

Re: [PATCH v12 0/9] MR1297: gdiplus: Add CustomLineCap implementation (including displaying).
by Esme Povirk (＠madewokherd) 21 Nov '22

21 Nov '22

Esme Povirk (@madewokherd) commented about dlls/gdiplus/graphicspath.c: > + widen_closed_figure(tmp_points, 0, custom->pathdata.Count - 1, pen, pen_width, last_point); > + else > + widen_open_figure(tmp_points, 0, custom->pathdata.Count - 1, pen, pen_width, LineCapFlat, LineCapFlat, last_point); > + } > + else > + { > + for (INT i = 0; i < custom->pathdata.Count; i++) > + { > + // rotation of CustomCap according to line > + perp_dx = custom->pathdata.Points[i].X * cosa + (custom->pathdata.Points[i].Y - 1.0) * sina; > + perp_dy = custom->pathdata.Points[i].X * sina - (custom->pathdata.Points[i].Y - 1.0) * cosa; > + *last_point = add_path_list_node(*last_point, posx + perp_dx, > + posy + perp_dy, custom->pathdata.Types[i]); > + } > + } > + // FIXME: The line should be shorter, and it should ends up when Arrow/Cap is started draw This comment makes things more confusing, I think. I'd suggest something like `FIXME: The line should be adjusted by the inset value of the custom cap.`. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/1297#note_16777

1 0

[PATCH v2 0/5] MR1472: opengl32: Complete wow64 thunks implementation.
by Rémi Bernon (＠rbernon) 21 Nov '22

21 Nov '22

There's a few missing thunks for some non-standard extension functions, but that probably doesn't matter. -- v2: opengl32: Use the +opengl debug channel everywhere. opengl32: Cast wow64 pointers in glGet(VertexAttrib)Pointerv. opengl32: Implement wow64 thunks for glMapBuffer (et al.). opengl32: Use manual win32 thunks for glMapBuffer (et al.). opengl32: Implement wow64 thunk for glGetString (et al.). https://gitlab.winehq.org/wine/wine/-/merge_requests/1472

3 7

[PATCH v2 0/13] MR1447: mshtml: Implement performance.timing timestamps properly.
by Gabriel Ivăncescu (＠insn) 21 Nov '22

21 Nov '22

-- v2: mshtml: Implement performance.timing.msFirstPaint. mshtml: Implement performance.timing.loadEventStart & loadEventEnd. mshtml: Get rid of useless window check. mshtml: Implement performance.timing.domContentLoadedEventStart & End. mshtml: Implement performance.timing.domComplete. mshtml: Implement performance.timing.domInteractive. mshtml: Implement performance.timing.domLoading. mshtml: Implement performance.timing.responseStart & responseEnd. mshtml: Implement the remaining pre-response performance.timing props. mshtml: Implement performance.timing.unloadEventStart & unloadEventEnd. mshtml: Implement performance.timing.redirectStart. mshtml: Implement performance.timing.navigationStart. mshtml: Store link to the inner window in HTMLPerformance. https://gitlab.winehq.org/wine/wine/-/merge_requests/1447

4 21

[PATCH v2 0/1] MR1455: dinput: Wait on event instead of thread on stop
by Fabian Maurer (＠DarkShadow44) 21 Nov '22

21 Nov '22

Since we stop the thread in DllMain, we always haev the loaderlock. Thread shutdown also requires the loader lock, so we will always time out So just wait until all cleanup is done, the thread itself can exit later Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=52314 Signed-off-by: Fabian Maurer <dark.shadow4(a)web.de> -- v2: dinput: Wait on event instead of thread on stop https://gitlab.winehq.org/wine/wine/-/merge_requests/1455

4 7

2025

2024

2023

2022

wine-gitlab