Re: [PATCH 1/4] mshtml: Don't free string that will be cleared by SafeArrayDestroy.

[Michael Stefaniuc]

Hello Jacek,

Jacek Caban wrote:
> dlls/mshtml/tests/dom.c |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)

in that case "str" becomes unused and can be removed like in the
attached patch.

bye
    michael

[PATCH 1/4] mshtml: Don't free string that will be cleared by SafeArrayDestroy.

[Jacek Caban]

Hi Michael,

On 11/8/10 12:04 PM, Michael Stefaniuc wrote:
> Hello Jacek,
>
> Jacek Caban wrote:
> > dlls/mshtml/tests/dom.c |    1 -
> >   1 files changed, 0 insertions(+), 1 deletions(-)
>
> in that case "str" becomes unused and can be removed like in the
> attached patch.

Good catch, thanks. I've resent the patch.

Thanks,
     Jacek