[FFmpeg-devel] [PATCH] lavfi: add fade filter

[Diego Biurrun]

On Mon, Feb 21, 2011 at 12:02:57PM +0100, Stefano Sabatini wrote:
> 
> Port fade filter from libavfilter soc repo, with minor fixes by
> Stefano.
> 
> --- a/doc/filters.texi
> +++ b/doc/filters.texi
> @@ -465,6 +465,39 @@ within the parameter list.
>  
> + at section fade
> +
> +Apply fade-in/out effect to input video.
> +
> +It accepts the parameters:

As a rule of thumb, avoid "it".  There are too many opportunities to
make a text harder to understand with it, while more specific sentence
subjects almost never hurt.

> + at var{type} specifies if the effect type, can be either "in" for
> +fade-in, or "out" for a fade-out effect.

s/if//?

> + at var{start_frame} specifies the number of the start frame for starting
> +to apply the fade effect.

var{start_frame} specifies the number of the frame at which to start
applying the fade effect.

> +A few usage examples follow, usable too as test scenarios.

A few usage examples that may also serve as test scenarios follow.

> + at example
> +# fade in first 30 frames of video

I'd properly capitalize here, same below.

Diego