[EMAIL PROTECTED] writes:

> This is another attempt to add IDispatch support to several 2-operand
> functions. Now the macro was removed and the VT_DISPATCH handling was
> copied to every affected function.

Thanks, it's better that way. Still, it seems to me you don't always
clean the variants properly, you should do some sort of "goto end" in
all cases instead of simply returning. Also renaming the function
parameters in most cases is not necessary. I'd also suggest to do a
separate patch for each function, especially since in some functions a
bit of restructuring will be needed to cleanup properly on error.

-- 
Alexandre Julliard
[EMAIL PROTECTED]


Reply via email to