[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: [PATCH] set last error on enumerating displaysettings
From:       "Dmitry Timoshkov" <dmitry () codeweavers ! com>
Date:       2007-12-31 6:21:50
Message-ID: 010b01c84b75$720e3650$a600fea9 () DMITRY
[Download RAW message or body]

"Marcus Meissner" <marcus@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.


[prev in list] [next in list] [prev in thread] [next in thread]