[PATCH 4/5] d3d10/effect: Fix iteration in GetBackingStore() for sampler states.

Nikolay Sivov nsivov at codeweavers.com
Thu Jan 20 08:22:22 CST 2022


On 1/20/22 16:38, Matteo Bruni wrote:
> Nice find. I guess it makes sense that they did the same thing with
> the state objects. Weird, but...

At least it's consistent so far.

>
> While looking at this I realized something mostly unrelated: I guess
> we're supposed to free the descriptors and return NULL from
> GetBackingStore() after Optimize() is called? No need to do anything
> about it right away in any case, besides maybe extending
> test_effect_optimize() to show the expected behavior.
I don't know about that, I haven't tried. We'll still need this storage 
for updates through expressions. My impression was that Optimize() 
strips "nonessential" parts, like strings/annotations only.



More information about the wine-devel mailing list