[2/2] gdi32/tests: Added tests for GetFontResourceInfoW
Dmitry Timoshkov
dmitry at baikal.ru
Fri Dec 14 02:04:28 CST 2012
Stefan Leichter <Stefan.Leichter at camline.com> wrote:
> > > + pCopyFileW = (void *)GetProcAddress(hkernel32, "CopyFileW");
> > > + pExpandEnvironmentStringsW = (void *)GetProcAddress(hkernel32,
> > > "ExpandEnvironmentStringsW"); + pGetCurrentDirectoryW = (void
> > > *)GetProcAddress(hkernel32, "GetCurrentDirectoryW");
> >
> > There is no reason to import these APIs using GetProcAddress. Also testing
> > GetFontResourceInfo without checking actual buffer contents is not very
> > useful.
>
> According to MSDN this APIs are WIn2k++ and WinXP++ only
That's not true. You can always send a test to testbot and see what happens.
> > Google points to http://www.undocprint.org/winspool/getfontresourceinfo
> > which seems to have some description of this API, with symbolic names of
> > the type parameter, so testing something besides type == 1 would be a good
> > idea as well.
>
> The tests show what needs to be implemented to fix bug #30514. Feel free to
> send tests for other query types ;-)
Adding such tests isn't too hard, but anyway please check at least contents of
the buffer you get, and use symbolic names for query types.
--
Dmitry.
More information about the wine-devel
mailing list