Guy Albertelli wrote:
Fixed memory leak and changed skip to win_skip in this patch.
Changelog: - Enhance QueryDosDevice test for attempting to return all devices. - Add tests for GetVolumeInformationA testing the different root dir formats. --- dlls/kernel32/tests/volume.c | 137 ++++++++++++++++++++++++++++++++++++++++- 1 files changed, 133 insertions(+), 4 deletions(-)
------------------------------------------------------------------------
One of the tests fails here: volume.c:290: Test failed: GetVolumeInformationA w/o '\' did not fail, last error 2 I just ran the tests as your second mail/patch suggested it fixed something. That patch however didn't contain a change to the tests. If you add tests that fail on Wine and you don't include a fixed implementation in the same patch you should mark these with todo_wine(). If you fix the implementation in a later patch you should remove the todo_wine() again if needed. Bottom line: after each patch all tests should still succeed. -- Cheers, Paul.