[FFmpeg-devel] [PATCH] avfilter/graphparser: allow specifying filter at id as filter instance

[Muhammad Faiz]

On Thu, May 18, 2017 at 11:48 PM, Paul B Mahol <onemda at gmail.com> wrote:
> On 5/18/17, Muhammad Faiz <mfcc64 at gmail.com> wrote:
>> On Thu, May 18, 2017 at 11:11 PM, Paul B Mahol <onemda at gmail.com> wrote:
>>> On 5/17/17, Muhammad Faiz <mfcc64 at gmail.com> wrote:
>>>> See http://lists.ffmpeg.org/pipermail/ffmpeg-user/2017-April/035975.html
>>>> Parsed_filter_X is not intuitive as filter instance name and
>>>> also undocumented.
>>>>
>>>> Signed-off-by: Muhammad Faiz <mfcc64 at gmail.com>
>>>> ---
>>>>  doc/filters.texi          | 13 ++++++++++---
>>>>  libavfilter/graphparser.c | 26 +++++++++++++++++++++-----
>>>>  2 files changed, 31 insertions(+), 8 deletions(-)
>>>>
>>>
>>> Could you add example in git log when you also linked that thread
>>> article.
>>
>> Added
>> Example:
>> ffplay -f lavfi "nullsrc=s=640x360,
>> sendcmd='1 drawtext at top reinit text=Hello; 2 drawtext at bottom reinit
>> text=World',
>> drawtext at top=x=16:y=16:fontsize=20:fontcolor=Red:text='',
>> drawtext at bottom=x=16:y=340:fontsize=16:fontcolor=Blue:text=''"
>>
>>>
>>> IMHO ParsedXXXX could remain and user can override it with custom one.
>>
>> Changed.
>>
>> Thank's
>
> No more comments from me.

Applied.

Thank's.