[vlc-devel] libvlc API: Add equalizer to audio.c
Pierre d'Herbemont
pdherbemont at gmail.com
Wed Jul 29 10:56:57 CEST 2009
On Jul 29, 2009, at 12:52 AM, jpd at videolan.org wrote:
> On Tue, Jul 28, 2009 at 09:42:22AM -0700, Pierre d'Herbemont wrote:
>> On Jul 31, 2009, at 5:38 AM, Cyril MATHE wrote:
>>> bool libvlc_audio_equalizer_get_enable(libvlc_audio_equalizer*)
>>> void libvlc_audio_equalizer_set_enable(libvlc_audio_equalizer*, bool
>>> b_enable)
>>
>> This shouldn't be needed. The equalizer should be enabled as soon as
>> media_player_set_equalizer(equlizer);
>> is called.
>>
>> I would say:
>> media_player_set_equalizer(NULL);
>> Which could be equivalent to:
>> media_player_set_equalizer(Libvlc_DisabledEqualizer);
>>
>> to disable it from a specified instance.
>
> I don't disagree but I wonder if it wouldn't be more interesting to
> make equalizer a filter so that you can put multiple in a chain if
> you have to. Say, one before transcoding and one after.
That would be much more interesting, but that's a different (and a bit
more complex) API.
I think what we have don't prevent adding a separate filter based API
in the future.
Pierre.
More information about the vlc-devel
mailing list