diff mbox

[FFmpeg-devel,v2] doc/demuxers: correct grammatical errors of m3u8_hold_counters option

Message ID 20191127060324.20443-1-lq@chinaffmpeg.org
State New
Headers show

Commit Message

Liu Steven Nov. 27, 2019, 6:03 a.m. UTC
Suggested-by: Rodney Baker <rodney.baker@iinet.net.au>
Signed-off-by: Steven Liu <lq@chinaffmpeg.org>
---
 doc/demuxers.texi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Gyan Doshi Nov. 27, 2019, 7:44 a.m. UTC | #1
On 27-11-2019 11:33 am, Steven Liu wrote:
> Suggested-by: Rodney Baker <rodney.baker@iinet.net.au>
> Signed-off-by: Steven Liu <lq@chinaffmpeg.org>
> ---
>   doc/demuxers.texi | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/doc/demuxers.texi b/doc/demuxers.texi
> index 4e1a5cb6aa..0d13bdd1b3 100644
> --- a/doc/demuxers.texi
> +++ b/doc/demuxers.texi
> @@ -332,7 +332,7 @@ Maximum number of times a insufficient list is attempted to be reloaded.
>   Default value is 1000.
>   
>   @item m3u8_hold_counters
> -Maximum number of times to load m3u8 when the m3u8 dose not refresh with new segments.
> +The maximum number of times to load m3u8 when it refreshes without new segments.
>   Default value is 1000.
>   
>   @item http_persistent

LGTM.

Suggest you change the description to this in the AVOption as well.

Gyan
Liu Steven Nov. 27, 2019, 11:05 a.m. UTC | #2
> 在 2019年11月27日,15:44,Gyan <ffmpeg@gyani.pro> 写道:
> 
> 
> 
> On 27-11-2019 11:33 am, Steven Liu wrote:
>> Suggested-by: Rodney Baker <rodney.baker@iinet.net.au>
>> Signed-off-by: Steven Liu <lq@chinaffmpeg.org>
>> ---
>>  doc/demuxers.texi | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/doc/demuxers.texi b/doc/demuxers.texi
>> index 4e1a5cb6aa..0d13bdd1b3 100644
>> --- a/doc/demuxers.texi
>> +++ b/doc/demuxers.texi
>> @@ -332,7 +332,7 @@ Maximum number of times a insufficient list is attempted to be reloaded.
>>  Default value is 1000.
>>    @item m3u8_hold_counters
>> -Maximum number of times to load m3u8 when the m3u8 dose not refresh with new segments.
>> +The maximum number of times to load m3u8 when it refreshes without new segments.
>>  Default value is 1000.
>>    @item http_persistent
> 
> LGTM.
> 
> Suggest you change the description to this in the AVOption as well.
fixed locally and pushed. commit id: afee801276d5aea66057aa9d4d37a13cf7ae7372
> 
> Gyan
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Thanks
Steven
diff mbox

Patch

diff --git a/doc/demuxers.texi b/doc/demuxers.texi
index 4e1a5cb6aa..0d13bdd1b3 100644
--- a/doc/demuxers.texi
+++ b/doc/demuxers.texi
@@ -332,7 +332,7 @@  Maximum number of times a insufficient list is attempted to be reloaded.
 Default value is 1000.
 
 @item m3u8_hold_counters
-Maximum number of times to load m3u8 when the m3u8 dose not refresh with new segments.
+The maximum number of times to load m3u8 when it refreshes without new segments.
 Default value is 1000.
 
 @item http_persistent