[FFmpeg-devel] [PATCHv2] doc/demuxers: add some concat demuxer script examples

Tobias Rapp t.rapp at noa-archive.com
Wed Feb 3 11:51:16 CET 2016


On 03.02.2016 09:48, Paul B Mahol wrote:
> On 2/3/16, Tobias Rapp <t.rapp at noa-archive.com> wrote:
>> On 26.01.2016 09:09, Tobias Rapp wrote:
>>> On 25.01.2016 13:22, Nicolas George wrote:
>>>> Le decadi 30 nivose, an CCXXIV, Tobias Rapp a ecrit :
>>>>> Attached patch adds some example scripts for the concat demuxer to the
>>>>> documentation.
>>>>
>>>> Well, I maintain the code, not really the documentation.
>>>>
>>>>> >From 5ffc11e8139476d18cd2eaa28338adb0dda80999 Mon Sep 17 00:00:00 2001
>>>>> From: Tobias Rapp <t.rapp at noa-archive.com>
>>>>> Date: Tue, 19 Jan 2016 15:42:33 +0100
>>>>> Subject: [PATCH] doc/demuxers: add some concat demuxer script examples
>>>>>
>>>>> Signed-off-by: Tobias Rapp <t.rapp at noa-archive.com>
>>>>> ---
>>>>>    doc/demuxers.texi | 21 +++++++++++++++++++++
>>>>>    1 file changed, 21 insertions(+)
>>>>>
>>>>> diff --git a/doc/demuxers.texi b/doc/demuxers.texi
>>>>> index fb1e4fb..3900272 100644
>>>>> --- a/doc/demuxers.texi
>>>>> +++ b/doc/demuxers.texi
>>>>> @@ -214,6 +214,27 @@ The default is 0.
>>>>>
>>>>>    @end table
>>>>>
>>>>> + at subsection Examples
>>>>> +
>>>>
>>>>> +Example script which uses absolute filenames and includes some
>>>>> comments:
>>>>
>>>> I am not sure that "which" is the most idiomatic here, "that" sound
>>>> better,
>>>> but I am not a native speaker.
>>>
>>> I can also avoid which/that by "itemize"-ing the examples similar to
>>> other example sections within the documentation. Attached an updated
>>> version of the patch.
>>>
>>>> [...]
>>
>> Ping.
>>
>
> It should be fine if examples have been tested to actually work.

Oops. You're right the second example doesn't work in safe mode because 
whitespace is not allowed. Have sent version 3 of the patch in a 
separate mail. Now the two examples have been tested literally.

Regards,
Tobias



More information about the ffmpeg-devel mailing list