[Bug 37118] PVS-Studio Error: Typos

wine-bugs at winehq.org wine-bugs at winehq.org
Sat Aug 16 13:50:48 CDT 2014


https://bugs.winehq.org/show_bug.cgi?id=37118

--- Comment #1 from Nikolay Sivov <bunglehead at gmail.com> ---
> V501 There are identical sub-expressions to the left and to the right of the 
> '|' operator: VT_ARRAY | VT_ARRAY vartest.c 2161

I have a patch for that one and some others.

> if ((hr = SafeArrayAccessData( sa, (void **)&ptr )) != S_OK)
>  return hr;
> if ((hr = SafeArrayGetUBound( sa, 1, &size ) != S_OK)) //<==
> {
>    SafeArrayUnaccessData( sa );
>    return hr;
>}

That's a valid one too. Looking at it now.

Please don't mix separate problems in a same report, stick to the rule of 1
problem per bug report.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.



More information about the wine-bugs mailing list