On 23 September 2012 22:40, Ričardas Barkauskas <[email protected]> wrote: > > +struct refresh_rates { Formatting.
> +struct refresh_rates refresh_list[] = This should be static const. > + ok(hr == E_INVALIDARG, "GetDesc unexpectedly returend %#x.\n", hr); "returned". > + for (i = 0; i < sizeof(refresh_list)/sizeof(struct refresh_rates); i++) I generally prefer "sizeof(*refresh_list)", or perhaps "sizeof(refresh_list[0])" over "sizeof(struct refresh_rates)" in this kind of situation. > + "Numerator %d is %d.\n", i, > result_desc.BufferDesc.RefreshRate.Numerator); These are unsigned variables.
