On Sat, Jul 5, 2008 at 9:24 PM, Vitaliy Margolen wine-patches@kievinfo.com wrote: <snip>
LinuxInputEffectImpl_Stop(iface);LinuxInputEffectImpl_Unload(iface);
<snip>
Vitaliy,
Native behavior is for Unload to stop the effect also (see msdn or test it yourself), so the call to Stop here is unnecessary. If the call to Stop is required, it must be done in Unload, not Release, as it's not unusual to unload an effect without releasing it when a game has a large number of available effect instances.
It is my understanding that no explicit call to Stop is needed, either in Release or Unload, as evdev has the same behavior documented (see section 3.3 of ff.txt in the kernel docs).
The call to Unload is certainly correct, and I don't think there's anything strictly wrong with calling Stop; just make sure it's not covering up a bug in Unload.
Regards, Daniel Remenak