[shell32tests/shelllink.c] Use aliases for ordinals (resend/rediff)
Francois Gouget
fgouget at codeweavers.com
Wed Feb 16 12:53:54 CST 2005
Paul Vriens wrote:
> Hi,
>
> to ease Alexandre's work I re-diffed the patch. The consensus on
> wine-devel was that this patch is fine.
>
> This patch makes the crosstest compile work again for shell32. Result
> tested on win98/winxp/Wine/W2KProf.
>
> Changelog:
> Use aliases for calls to ordinals.
I don't think that's what the consensus says, if a consensus there is in
the first place.
If I understand Filip Navara's patch correctly, it shows that the fix
belongs in MinGW's shell32.def file. And if that's the case then I'm
arguing this test should remain as is.
As I understand it, once MinGW's file is fixed, the compiler sees that
there is an ordinal associated to SHSimpleIDListFromPath and then
generates an executable which imports SHSimpleIDListFromPath by ordinal.
This seems to be exactly what Visual C++ does on Windows.
It's perfectly acceptable to import SHSimpleIDListFromPath by name in
programs compiled using Visual C++, and the resulting executable works
even if shell32.dll exports this API by ordinal only. Our conformance
tests should reflect that. That's because they are meant to reflect the
Windows behavior (not the MinGW one), including where compilation and
linking are concerned.
--
Francois Gouget
fgouget at codeweavers.com
More information about the wine-patches
mailing list