On 3/1/19 3:00 PM, Gabriel Ivăncescu wrote:
On 3/1/19 3:53 PM, Andrew Eikum wrote:
On Thu, Feb 28, 2019 at 12:50:29AM +0100, Michael Stefaniuc wrote:
diff --git a/dlls/dsound/primary.c b/dlls/dsound/primary.c index 9f41a2b47f..852ec51b7f 100644 --- a/dlls/dsound/primary.c +++ b/dlls/dsound/primary.c @@ -576,7 +576,7 @@ static HRESULT WINAPI PrimaryBufferImpl_SetVolume(IDirectSoundBuffer8 *iface, LO for (i = 0; i < DS_MAX_CHANNELS; i++) { if (device->pwfx->nChannels > i){ - fvol = (float)((DWORD)(device->volpan.dwTotalAmpFactor[i] & 0xFFFF) / (float)0xFFFF); + fvol = (float)((device->volpan.dwTotalAmpFactor[i] & 0xFFFF) / (float)0xFFFF); hr = IAudioStreamVolume_SetChannelVolume(device->volume, i, fvol); if (FAILED(hr)){ LeaveCriticalSection(&device->mixlock);
I guess I'm not opposed to these patches, but I don't really see the benefit either.
Andrew
Sorry if I intrude, but this has me wondering now. Should pointless casts not be removed, in general? Obviously there's no benefit other than cleaning the code. (I'm talking in general wine policy, not this specific code).
I do that on and off since 2007. I removed a few thousands of those :)
bye michael