Although the test has never been observed to fail on Windows, I think the
failure is genuinely a race in the test.
File handles (like events) are signaled in order to mark that an I/O operation
has completed. In this case the I/O operation includes manipulating data on both
ends of the pipe, and as part of that may signal the other end. Internally,
however, the file handle must logically happen *after* all of this processing
has taken place, not least because (given the Windows I/O architecture) it is
the job of the I/O manager, not the IRP handler.
Since the purpose of the test is probably just to check that the file handle
will be signaled after a synchronous I/O operation has completed, we may as well
check it from the client thread, after we know for sure that it has.
Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=54078
This reverts commit 4608e1e125.
The ASF Reader filter suffers from defects which are difficult to solve,
and will require a better implementation of wmvcore in order to work
correctly. In the meantime, revert the class registration so that the
File Source filter is preferred instead.
This will break games that tried to use the filter directly and which
may have started to work correctly, but they are less frequent than the
ones simply playing wmv videos with quartz.
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53640
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53748
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54191
We get conhost.exe:tty failures on testbot:
- not for version prior to w1064v1909 (these tests require
pseudo-consoles, which are not implemented in these Windows
version).
- for w1064v1909
- not for W10 versions after w1064v1909.
So, we assume that pseudo-console support is immature in this
Windows version: skip all the remaining test at the first
discrepancy point.
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53465
Signed-off-by: Eric Pouech <eric.pouech@gmail.com>
Instead of uploading all currently bound textures. Besides being redundant, this
would generate mipmaps for an uninitialized texture, and subsequently trying to
render from nonzero mipmap levels would yield uninitialized data.
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54170
udev_add_device declares a variable (product) even when it is not used
on all platforms. Use the same condition for the declaration that already
guards the code.
init_argv0_dir has conditional code for different operating systems. In
case of FreeBSD a variable remains uninitialized in the error case, yet
is then used. Fix that by handling the error case.
WriteConsole (not in processed mode) and WriteConsoleOutput* functions
allow to write control characters, which can then be read back as is.
Signed-off-by: Eric Pouech <eric.pouech@gmail.com>