A temporary variable is used here to assist with assignment
but this does not conform to the coding style in the rest of
gdiplus and introduces an unnecessary variable.
Signed-off-by: David Kahurani <k.kahurani@gmail.com>

Signed-off-by: Eric Pouech <epouech@codeweavers.com>

Signed-off-by: Eric Pouech <epouech@codeweavers.com>

When the malloc for len_namespaces should fail namespaces
would be uninitialized and passed to free.

Signed-off-by: Eric Pouech <epouech@codeweavers.com>

Signed-off-by: Eric Pouech <epouech@codeweavers.com>

Wine-Bug: https://bugs.winehq.org//show_bug.cgi?id=55491

They are constantly failing on Gitlab, and may be caused various race
conditions around window focus, both in Wine and in Fvwm. It would be
nice to fix them but it's been going for many years now and unlikely to
be the case soon.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56119

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=56168Signed-off-by: Eric Pouech <epouech@codeweavers.com>

This will now fail on older, unsupported versions, but the
test_monochrome_icon test would be pointless otherwise.

They most of the time let any result pass and defeat the purpose of
the test.