[PATCH] d3dx9: IsParameterUsed stub should return TRUE.

Paul Gofman gofmanp at gmail.com
Mon Mar 7 17:22:07 CST 2016


On 03/08/2016 01:42 AM, Matteo Bruni wrote:
>
>>> a proper implementation would be even better (although it may be non
>>> obvious to find out if a parameter is used, depending on what "used"
>>> really means - it needs tests)...
>> You always need tests, but a reasonable guess would be whether the
>> parameter is referenced by the technique.
> That's likely to be the case, but yeah - tests.
>
>
I actually already got real implementation for this function (here
inside a big & dirty cumulative patch
https://bugs.winehq.org/attachment.cgi?id=53869), but this requires
testing as you say.
My motivation to change the stub at this point was that it currently
preventing testing the other bigger parts of effect framework on real
applications, and real implementation will be easier test and add later.
Since I can't add alltogether, I need to choose what to deal with first,
and I thought it might be easier to verify a bigger piece when I get
them working in real life rather then putting all the blocks in before
getting anything real working.





More information about the wine-devel mailing list