Re: [FFmpeg-devel] [PATCH v2 2/2] avformat/flvdec: support demux HEVC in Enhancing FLV

From: "Martin Storsjö" <martin@martin.st>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Cc: Steven Liu <lq@chinaffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH v2 2/2] avformat/flvdec: support demux HEVC in Enhancing FLV
Date: Thu, 6 Apr 2023 16:19:56 +0300 (EEST)
Message-ID: <e9aa20ad-1f69-2e6b-8a1-8f1426ac880@martin.st> (raw)
In-Reply-To: <CADxeRwm8_F2LeB9qo7fdChrK=fUe5Pmqhew+GQO2G5SyzQH3mQ@mail.gmail.com>

On Thu, 6 Apr 2023, Steven Liu wrote:

> Martin Storsjö <martin@martin.st> 于2023年4月6日周四 20:31写道：
>>
>> On Thu, 6 Apr 2023, Steven Liu wrote:
>>
>> > Signed-off-by: Steven Liu <lq@chinaffmpeg.org>
>> > ---
>> > libavformat/flvdec.c | 63 +++++++++++++++++++++++++++++++++++++++-----
>> > 1 file changed, 56 insertions(+), 7 deletions(-)
>>
>> Even if the title of the linked document is "Enhancing RTMP, FLV" I kinda
>> would expect the name of the concept/format to be more like "Enhanced FLV"
>> (as the name of the repo is "enhanced-rtmp", not "enhancing-rtmp").
>>
>> So I'd kinda prefer to refer to it in that form in the commit subject and
>> in the comment in the text. If you want to use literally the name used in
>> the document, I guess that's "Enhancing RTMP, FLV" though.

> Or submit three patches, BTW supprt the Enhanced rtmp?

I don't really understand what you mean here and how this is a response to 
my comment?

Please respond with a bit more elaborate answer.

// Martin
_______________________________________________
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".