[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