OLEAUT32: implement OleLoadPictureFile(Ex) and add tests

Detlef Riekenberg wine.dev at web.de
Fri Nov 14 14:17:07 CST 2008


On Mi, 2008-11-12 at 23:40 -0800, Jeremy Drake wrote:

I have no Idea about that area, but the unicode API is
not implemented on win9x
(GetTempFileNameW, CreateFileW, DeleteFileW)


>  if(!pOleLoadPictureFile || !pOleLoadPictureFileEx)
Are there systems in the Wild that have
OleLoadPictureFile, but are missing OleLoadPictureFile?
In that case, you should use 2 seperate tests.

> skip("Skipping OleLoadPictureFile tests\n");

Think, what you would see in the log before your text:
"filename.c:__LINE__: Tests skipped:"
"Skipping" is redundant and "tests" also.

The usual way would be:
 skip("OleLoadPictureFile not found\n");

When you want to make sure, that we never skip in Wine here,
then use win_skip instead.

-- 
 
By by ... Detlef





More information about the wine-devel mailing list