[Bug 39483] Logically dead code in stub_manager_new_ifstub (Coverity #731556)

[wine-bugs at winehq.org]

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

François Gouget <fgouget at codeweavers.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |source

--- Comment #5 from François Gouget <fgouget at codeweavers.com> ---
Sorry about the line number. So this code was added by the following commit:

commit 611b5acbf8263f14292eaca6be175b557c6a9d35
Author: Alexandre Julliard <julliard at winehq.org>
Date:   Fri Sep 2 11:18:45 2005 +0000

    Authors: Mike Hearn <mh at codeweavers.com>, Robert Shearman
<rob at codeweavers.com>
    Change stub manager to track the number of normal marshals instead of
    using the state machine so that multiple marshals of the same object
    and interface work correctly.

Interestingly Marcus already submitted a patch for this last may:
https://www.winehq.org/pipermail/wine-patches/2015-May/139094.html

His patch got rejected, probably because it causes tests/marshal.c to crash.
But I don't know anything about that code either so someone else will have to
figure it out.

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