[Wine-devel] [Wine-patches] [eter-2.0.0] [1/2] kernel32: Set NO_ERROR code. (eterbug #8821)

[Dmitry Timoshkov]

Dmitry Timoshkov <dtimoshkov на etersoft.ru> wrote:

> Tatyana Fokina <tatyana на etersoft.ru> wrote:
> 
> > +        SetLastError(NO_ERROR);
> >          MultiByteToWideChar(locale_cp, 0, str1, len1, str1W, len1W);
> >      }
> >      len2W = MultiByteToWideChar(locale_cp, 0, str2, len2, buf2W, 130);
> > @@ -2899,6 +2900,7 @@ INT WINAPI CompareStringA(LCID lcid, DWORD style,
> >              SetLastError(ERROR_NOT_ENOUGH_MEMORY);
> >              return 0;
> >          }
> > +        SetLastError(NO_ERROR);
> >          MultiByteToWideChar(locale_cp, 0, str2, len2, str2W, len2W);
> >      }
> 
> Очень сомнительный и скорее всего ненужный патч.

Перечитал объяснение в задаче 8821, и если действительно проблема в том,
что CompareStringA() не должна оставлять ошибку в ERROR_INSUFFICIENT_BUFFER
после успешного конвертирования строк длиннее 130 символов, то для этого случая
можно и нужно написать тест.

-- 
Dmitry.