On 09/28/2012 05:19 AM, Diego Biurrun wrote:
> On Fri, Sep 28, 2012 at 12:54:07AM -0400, Justin Ruggles wrote:
>> --- a/doc/filters.texi
>> +++ b/doc/filters.texi
>> @@ -322,30 +322,49 @@ conversion.
>>  
>> -The filter accepts exactly one parameter @var{vol}, which expresses
>> -how the audio volume will be increased or decresed.
>> +The filter accepts the following named parameters:
>> +@table @option
>> +
>> +@item volume
>> +Expresses how the audio volume will be increased or decreased.
>>  
>>  Output values are clipped to the maximum value.
>>  
>> -If @var{vol} is expressed as a decimal number, and the output audio
>> +If @var{volume} is expressed as a decimal number, and the output audio
>>  
>> -If @var{vol} is expressed as a decimal number followed by the string
>> +If @var{volume} is expressed as a decimal number followed by the string
> 
> I'd do s/expressed/given/ here to avoid repeating "expressed",
> which was used above.
> 
>> +@table @option
>> +@item fixed
>> +8-bit fixed-point. Limits input sample format to U8, S16, and S32.
>> +@item float
>> +32-bit floating-point. Limits input sample format to FLT. (default)
>> +@item double
>> +64-bit floating-point. Limits input sample format to DBL.
>> +@end table
> 
> I'd replace the periods by semicolons, i.e.
> 
>   8-bit fixed-point; limits input sample format to U8, S16, and S32.
> 
> which makes it read more natural to me.
> 
> Don't bother sending a fresh patch for these changes.

Fixed locally.

-Justin

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to