Re: [PATCH 0/1] MR6342: wscript: Implement Sleep

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about programs/wscript/host.c:
> static HRESULT WINAPI Host_Sleep(IHost *iface, LONG Time)
>  {
> -    WINE_FIXME("(%ld)\n", Time);

Please change it to `TRACE()` instead of removing. This makes debugging things easier.

The implementation looks good, but it would be nice to have a test, see `tests/run.js`. Something like a simple `Sleep(0)` call and maybe a check that negative values throw an exception (see existing try..catch tests for an example).

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/6342#note_79649