On Tue Jan 23 16:39:13 2024 +0000, Esme Povirk wrote:
> Now that I'm looking closer, I'm even more confused. As far as I can
> tell from the tests, the parameter size of RedEyeCorrectionEffect should
> always be sizeof(RedEyeCorrectionEffect). So, why does
> GdipGetEffectParameterSize return something different?
> And if a different size is allowed to be passed in, there should be
> tests for it.
I have tested it, it always includes the size of the numareas. I have tested it, it throws InvalidParameter errors.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/4661#note_58668
On Tue Jan 23 16:34:02 2024 +0000, Esme Povirk wrote:
> I don't see anything changed that would address this.
Now that I'm looking closer, I'm even more confused. As far as I can tell from the tests, the parameter size of RedEyeCorrectionEffect should always be sizeof(RedEyeCorrectionEffect). So, why does GdipGetEffectParameterSize return something different?
And if a different size is allowed to be passed in, there should be tests for it.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/4661#note_58667
On Mon Jan 22 20:37:06 2024 +0000, Vijay Kiran Kamuju wrote:
> I have changed the structure of the CGpEffect and with it some
> functionality. Please re-check.
I don't see anything changed that would address this.
--
https://gitlab.winehq.org/wine/wine/-/merge_requests/4661#note_58666
--
v2: vkd3d-shader/dxil: Use strcmp() to find the handle type.
vkd3d-shader/dxil: Use strcmp() to find function names.
vkd3d-shader/dxil: Use strcmp() to check the entry point name.
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/591
--
v3: tests: Add tests for valid conditional types.
vkd3d-shader/hlsl: Always cast to bool in if() statements.
vkd3d-shader/hlsl: Validate that condition expressions are numeric.
vkd3d-shader/hlsl: Validate the condition data type for loops as well.
vkd3d-shader/hlsl: Copy some missing fields in hlsl_type_clone().
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/577
Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com>
--
v4: vkd3d-shader/fx: Add initial support for writing passes for fx_2_0.
vkd3d-shader/fx: Add initial support for writing fx_2_0 binaries.
vkd3d-shader/fx: Check technique type in global scope as well.
vkd3d-shader/hlsl: Allow annotations on techniques.
vkd3d-shader/hlsl: Allow annotations on passes.
vkd3d-shader/hlsl: Add passes variables to the techniques.
vkd3d-shader/hlsl: Add initial support for parsing annotations.
tests/hlsl: Add some tests for annotations.
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/565
~~Other validation stuff I plan to send as soon as !450 and !550 are in.~~
~~I guess the CI is going to timeout because of too many commits, so I'll preemptively stop it.~~
--
v9: vkd3d-shader/ir: Validate PHI instructions.
vkd3d-shader/ir: Do not allow IMMCONST and IMMCONST64 as destination registers.
vkd3d-shader/ir: Refactor register-type-specific code in parameter validation.
vkd3d-shader/ir: Check that all instructions appear in a block.
https://gitlab.winehq.org/wine/vkd3d/-/merge_requests/559