[FFmpeg-devel] [PATCH] avfilter: add audio pulsator filter

[Ganesh Ajjanagadde]

On Sun, Nov 29, 2015 at 12:45 PM, Clément Bœsch <u at pkh.me> wrote:
> On Sun, Nov 29, 2015 at 06:39:02PM +0100, Moritz Barsnick wrote:
>> Hi,
>>
>> On Sat, Nov 28, 2015 at 23:26:44 +0100, Paul B Mahol wrote:
>>
>> In addition to Ganesh's comments:
>>
>> > + at item amount
>> > +Set modulation. Define how much of original signal is affected by the LFO.
>> [...]
>> > + at item width
>> > +Set pulse width.
>>
>> I would appreciate ranges for both od these. I would guess them being
>> [0..1] (and I coould read the code), but who knows.
>>
>
> or do ffmpeg -h filter=apulsator

I thought the common idiom was to document these ranges twice, i.e in
both the docs and implicitly in the code via the options table?

>
> --
> Clément B.
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>