Re: [PATCH] inetcomm/smtptransport.c : Remove some unneeded variables and assigns.

[Amine Khaldi]

> Hi Amine,

Hi Juan,
> this patch has no functional benefit.  For example,
> -
>
> HRESULT hr;
>
> TRACE("\n");
> -
>
> hr = SMTPTransport_ParseResponse(This, pBuffer,&response);
> -
>
> if (FAILED(hr))
> - if (FAILED(SMTPTransport_ParseResponse(This, pBuffer,&response)))
>
> This has the dubious benefit of removing a local variable that was
> used before.  I can't fathom why this is a good thing.

It makes the code use one less variable. I don't see how is this 
dubious, but I understand that it's trivial.

WBR,
Amine.