[vlc-devel] [RFC PATCH 1/2] mmdevice: reflect the state of Sndvol slider

Thomas Guillem thomas at gllm.fr
Fri Apr 6 13:05:37 CEST 2018



On Fri, Apr 6, 2018, at 11:56, Thomas Guillem wrote:
> FYI:
>  - PowerDVD and MS Movies are doing like this patch (linear relation 
> between slider and ISimpleAudioVolume)

And Firefox too is doing like ^^

Tested with a 1kHZ sample. The loudness change sensation feel way more natural that way. It allows to do finer loudness changes with high percentage value.

>  - MPC-HW and Itunes are doing like current VLC (x³ relation between 
> slider and ISimpleAudioVolume)
> 
> I guess there is no wrong/good answer. Personally, I really prefer doing 
> the MS Movies way.
> 
> On Thu, Apr 5, 2018, at 17:35, Rémi Denis-Courmont wrote:
> > Le torstaina 5. huhtikuuta 2018, 12.24.13 EEST Thomas Guillem a écrit :
> > > Then, we have 4 choices:
> > > 
> > >  - a/ Keep the code as it is. I don't like it because it doesn't respect the
> > > Sndvol slider, and the first 60% on the slider doesn't change the loudness
> > > much (very hard to hear an audible difference with low % values).
> > > 
> > >  - b/ Do what this patch is doing. the relation between the slider and the
> > >    loudness is still not linear but it respect the Sndvol slider. I tested
> > > with few samples: changing volume feels way more natural (the loudness
> > > change a lot between 0 and 50% and much less after 90%).
> > 
> > Err, AFAIK, to respect the Windows mixer slider, you have to do that *and* 
> > restrict the volume range to 0-100%.
> > 
> > -- 
> > 雷米‧德尼-库尔蒙
> > https://www.remlab.net/
> > 
> > _______________________________________________
> > vlc-devel mailing list
> > To unsubscribe or modify your subscription options:
> > https://mailman.videolan.org/listinfo/vlc-devel
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list