[PATCH 5/5] kernel32/tests: Add a IOCTL_DVD_READ_STRUCTURE (DvdManufacturerDescriptor) test
GOUJON Alexandre
ale.goujon at gmail.com
Sun Jun 17 05:39:50 CDT 2012
On 06/17/2012 01:19 PM, Nikolay Sivov wrote:
> On 6/17/2012 13:11, Alexandre Goujon wrote:
>> case DvdManufacturerDescriptor:
>> + nt_desc.manf->Header.Length = 0x0802;
>> + nt_desc.manf->Header.Reserved[0] = 0;
>> + nt_desc.manf->Header.Reserved[1] = 0;
> Does this length mean something? Looks a bit dirty to leave hardcoded
> things like that, but maybe I don't know details.
This was hard-coded for the physical descriptor.
I tested it with the (few) DVD's I have at home.
But that's what tests are for : let (more) people check the behaviour
with more hardware/software/CDs/DVDs (than the developer has)
Anyway, thanks for the review (especially a Sunday)
More information about the wine-devel
mailing list