[2/2] gdi32/tests: Added tests for GetFontResourceInfoW (try 2)

Dmitry Timoshkov dmitry at baikal.ru
Fri Dec 28 22:53:37 CST 2012


Stefan Leichter <Stefan.Leichter at camline.com> wrote:

> First of all: The issues your are pointing out have been in the previous 
> version of the patch too. Why didn't you complain in your first review. To my 
> understanding this is a wast of my time complaining about two issues and 
> keeping the rest secret for the next version of the patch.

It's often happens with subsequent reviews, and there is nothing magic or
secret about it: first review was about the whole approach while next one
is more about the actual details.

> Now to the technical stuff:
> - i agree to change the data type to WCHAR
> - what you mean with "protect from insufficient buffer size" it totaly unclear to 
> me

I intentionally left 'BYTE  buffer[MAX_PATH];' in the quote, think about it.

> - the tests are written for a font in %windor%\fonts. This directory is empty 
> after a clean .wine directory was created. Changing the font does not help at 
> all.

But it will once Wine starts to copy its own fonts there at some point.

-- 
Dmitry.



More information about the wine-devel mailing list