April 2024 - Wine-gitlab - list.winehq.org

[PATCH 0/4] MR766: tests: Add CreateShaderCacheSession pointer clearing tests.
by Stefan Dösinger 10 Apr '24

10 Apr '24

Native's behavior is inconsistent. It sets the pointer to NULL when it fails to open an existing cache due to a version mismatch, but leaves the pointer untouched when the description fails validation. -- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/766

2 8

[PATCH v3 0/8] MR5459: mfreadwrite: Implement D3D awareness support with the video processor.
by Rémi Bernon 10 Apr '24

10 Apr '24

-- v3: mfreadwrite/reader: Pass the device manager to the stream transforms. winegstreamer/video_processor: Implement D3D awareness. mf/tests: Test video processor D3D11 awareness. mfreadwrite/tests: Add some source reader D3D11 awareness tests. mfreadwrite/reader: Avoid leaking the stream transform service MFT. mfreadwrite/tests: Shutdown the test stream event queues on source shutdown. mfreadwrite/tests: Avoid using MFCreateMediaBufferFromMediaType. mfreadwrite/tests: Do not accept MFVideoFormat_RGB32 in the test transform. https://gitlab.winehq.org/wine/wine/-/merge_requests/5459

2 9

[PATCH v5 0/5] MR767: vkd3d-shader/dxil: Tessellation part 1.
by Conor McCarthy (＠cmccarthy) 10 Apr '24

10 Apr '24

Doesn't support tessellation yet, but adds some requirements. -- v5: vkd3d-shader/ir: Validate tessellation declarations. vkd3d-shader/dxil: Implement DX intrinsic PrimitiveID. vkd3d-shader/dxil: Load hull shader properties. vkd3d-shader/dxil: Load domain shader properties. tests/hlsl: Add a tessellation test. https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/767

2 5

[PATCH v18 0/3] MR734: vkd3d-shader/dxil: SamplePosition and SampleCount.
by Conor McCarthy (＠cmccarthy) 10 Apr '24

10 Apr '24

-- v18: vkd3d-shader/dxil: Implement DX intrinsic RenderTargetGetSampleCount. vkd3d-shader/dxil: Implement DX intrinsics Texture2DMSGetSamplePosition and RenderTargetGetSamplePosition. tests/hlsl: Add tests for GetSamplePosition() and GetSampleCount(). https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/734

2 3

[PATCH 0/5] MR775: hlsl: Avoid HLSL_CLASS_OBJECT, part 3.
by Zebediah Figura (＠zfigura) 10 Apr '24

10 Apr '24

-- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/775

2 5

Re: [PATCH v3 0/1] MR761: tests/shader_runner: Report compilation errors on the block's starting line.
by Francisco Casas (＠fcasas) 09 Apr '24

09 Apr '24

On Tue Apr 9 22:59:29 2024 +0000, Henri Verbeet wrote: > > > This seems a little fragile. > > > > I don't think it is that fragile, currently we handle directives in 3 > points in run_shader_tests(): > > > > 1. The first switch, where we detect that we stopped reading the block > because a new block starts or the file ended. > > 2. The reading of the tags at the beginning of new blocks. > > 3. The reading of individual lines using a second switch. > > > > We don't need to call update_line_number_context() in 1, and we always > have to call it in 2 so we don't have to worry about forgetting to do so > we add new functionalities. > > > > It is only in 3 that we have to remember to call > update_line_number_context() for the switch cases when we want to update > the line number context for each line instead of presenting it at the > beginning of the block, but even if we forget, the error will be > reported at the beginning of the block because of 2. > Sure, the impact of messing up isn't going to very bad. Fundamentally > though, this commit puts the burden of calling > update_line_number_context() on the "normal" cases instead of on the > exceptional cases. > > > Could we just track the starting line for things touching > "shader_source" and then setup/restore the test context in e.g. compile_shader()? > > > > Here is a patch for that proposal (how I see it), but I find it more complex. > > > > [0001-tests-shader-runner-Report-compilation-errors-on-the.patch](https://gi… > I think that's closer to what I had in mind. How do you feel about [this patch](/uploads/e4b238bc14f76573c5eea824d660402f/patch.diff)? While that patch moves the burden to the _exceptional_ cases, with it we have to do many things for them now: - Set `current_source.start_line = runner->current_line` on 2. - Call `update_line_number_context(runner, source->start_line)` before performing the operations. - Call `update_line_number_context(runner, runner->current_line)` after performing the operations. - Call `clear_shader_source(&current_source)` after the source is stored in the `runner->XX_source` field I still think that only remembering to call `update_line_number_context()` on _normal_ cases is simpler, and not so hard to forget. When implementing new cases, at least I, take a look to other cases in the same switch for reference. -- https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/761#note_67443

1 0

Re: [PATCH v2 0/1] MR770: tests/test-driver: Avoid "head" with negative line count. - approved
by Alexandre Julliard (＠julliard) 09 Apr '24

09 Apr '24

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

1 0

Re: [PATCH v6 0/1] MR762: ci: Delimit building each commit with a collapsible section. - approved
by Alexandre Julliard (＠julliard) 09 Apr '24

09 Apr '24

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

1 0

Re: [PATCH v2 0/1] MR763: vkd3d-shader/hlsl: Properly release string buffer on write_atan_or_atan2(). - approved
by Alexandre Julliard (＠julliard) 09 Apr '24

09 Apr '24

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

1 0

Re: [PATCH v5 0/8] MR757: hlsl: Avoid HLSL_CLASS_OBJECT, part 2. - approved
by Alexandre Julliard (＠julliard) 09 Apr '24

09 Apr '24

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

1 0