[PATCH v2 1/3] ddraw/tests: Test non-primary surface in test_lost_device.

Gabriel Ivăncescu gabrielopcode at gmail.com
Mon Jul 27 12:36:19 CDT 2020


On 27/07/2020 19:30, Stefan Dösinger wrote:
> 
> 
>> Am 27.07.2020 um 19:05 schrieb Gabriel Ivăncescu 
>> <gabrielopcode at gmail.com <mailto:gabrielopcode at gmail.com>>:
>>
>> +    surface_desc.dwFlags = DDSD_CAPS | DDSD_WIDTH | DDSD_HEIGHT | 
>> DDSD_PIXELFORMAT;
>> +    surface_desc.ddsCaps.dwCaps = DDSCAPS_OFFSCREENPLAIN;
> 
> This will either create a video or system memory surface depending on 
> the whims of the driver. Since we already explicitly test offscreen 
> sysmem surfaces, shouldn't this one request video memory explicitly and 
> skip some tests if creation fails? That might make your broken(hr == 
> DD_OK) /* WinXP */ go away.
> 

You're right, sorry, I'm not very familiar with ddraw. I tried with 
DDSCAPS_VIDEOMEMORY (and even DDSCAPS_LOCALVIDMEM, though it shouldn't 
be necessary), unfortunately it's still broken on Windows XP so that 
broken check has to stay.



More information about the wine-devel mailing list