[PATCH] set last error on enumerating displaysettings

Dmitry Timoshkov dmitry at codeweavers.com
Mon Dec 31 00:21:50 CST 2007


"Marcus Meissner" <marcus at jet.franken.de> wrote:

> +    num = 1;
> +    while (1) {
> + SetLastError (0xdeadbeef);
> +    if (!EnumDisplaySettings(NULL, num++, &devmode)) {
> + DWORD le = GetLastError();
> + ok (le == ERROR_NO_MORE_FILES, "Last error on EnumDisplaySettings was %d, expected ERROR_NO_MORE_FILES\n", le);
> + break;
> + }
> +    }

Shouldn't the loop start from 'num = 0;' since the modes are 0 based?

-- 
Dmitry.



More information about the wine-devel mailing list