> > There seems to be some inconsistency between windows versions, the test

> > fails for xp and 2003. Should I just check for windows version and

> > adjust the tests accordingly?

>

> If it's not essential to demonstrate new behavior of version 6

> implementation, you can as well drop it.

It's not, I'll get rid of it when reducing the amount of tests.

> > +#define COMBO_YBORDERSIZE() 2

>

> Why is it defined like that if it's a constant? Also if it's going to be

> used in this new test only you can use actual constant variable in new

> test function.

It was defined in original combo like that, that's why I used that. But sure, I'll make it into a local constant.

> I'm a bit concerned about somewhat long loops this data will cause. Is

> it really necessary to have that many items?

Probably not, no. I just wanted to make sure, I'll trim that down.

> Please reformat this to use consistent spacing - 1 space after keyword.

Ah, sorry I always forget about that... I'll clean it up right away.

Thanks for the quick reply, I'll send an updated version soon.

Regards,

Fabian Maurer