From: Michael Stefaniuc Subject: urlmon/tests: Visually separate two if statements (PVS-Studio) Message-Id: <20141031095925.GA1463@redhat.com> Date: Fri, 31 Oct 2014 10:59:25 +0100 --- Using else-if would have worked too as gcc generates the same object file in both cases. dlls/urlmon/tests/protocol.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/dlls/urlmon/tests/protocol.c b/dlls/urlmon/tests/protocol.c index 9ffa04d..df3c364 100644 --- a/dlls/urlmon/tests/protocol.c +++ b/dlls/urlmon/tests/protocol.c @@ -1991,7 +1991,9 @@ static HRESULT WINAPI ProtocolEmul_Read(IInternetProtocolEx *iface, void *pv, memcpy(pv, "gif87a", 6); prot_read += *pcbRead = cb; return S_OK; - }if(prot_state == 4) { + } + + if(prot_state == 4) { *pcbRead = 0; return S_FALSE; } -- 1.9.3