Re: [2/2] mshtml: Add VT_INT support in IHTMLElementCollection_item

2008-12-27 Thread Jacek Caban
Hi Konstantin, Konstantin Kondratyuk wrote: Hi, Jacek! On Friday 19 December 2008 18:04:45 Jacek Caban wrote: You're duplicating the code that handles VT_I4 case. Please avoid it. Do you think, this code will be better? And can I truncate "VT_I4" from trace? Yes, it looks better

Re: [2/2] mshtml: Add VT_INT support in IHTMLElementCollection_item

2008-12-22 Thread Konstantin Kondratyuk
Hi, Jacek! On Friday 19 December 2008 18:04:45 Jacek Caban wrote: > You're duplicating the code that handles VT_I4 case. Please avoid it. Do you think, this code will be better? And can I truncate "VT_I4" from trace? if(V_VT(&name) == VT_I4 || V_VT(&name) == VT_INT) { int i;

Re: [2/2] mshtml: Add VT_INT support in IHTMLElementCollection_item

2008-12-19 Thread Jacek Caban
HI Konstantin, Konstantin Kondratyuk wrote: > VT_INT type isn't documented in MSDN, but it is supported in Windows > > You're duplicating the code that handles VT_I4 case. Please avoid it. Jacek