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

[Juan Lang]

Hi Amine,

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.
--Juan