- wine-gitlab - winehq.org

[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

[PATCH v3 0/1] MR1449: msado15: Avoid uninitialized variable warning in fields_get_Item
by Gerald Pfeifer (＠gerald) 21 Nov '22

21 Nov '22

This avoids the following warning by GCC 13: dlls/msado15/recordset.c:790:32: warning: 'i' may be used uninitialized -- v3: msado15: Avoid uninitialized variable warning in fields_get_Item https://gitlab.winehq.org/wine/wine/-/merge_requests/1449

4 3

2025

2024

2023

2022

wine-gitlab