Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-02 Thread Zhao Zhili
> On Jul 2, 2021, at 3:21 AM, Martin Storsjö wrote: > > On Thu, 1 Jul 2021, Jan Ekström wrote: > >> On Thu, Jul 1, 2021 at 5:16 PM James Almer wrote: >>> >>> From: Matthieu Patou >>> >>> Suggested-by: ffm...@fb.com >>> Signed-off-by: James Almer >>> --- >> >> For the record, if Big Compa

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-01 Thread libav
On 2021-07-01 12:58, Derek Buitenhuis wrote: On 7/1/2021 5:25 PM, Thomas Volkert wrote: I was also about to prepare a discussion mail against this patch set. We shouldn't fuzzy the spec. So.. +1 for skipping this patch set. Hard NAK from me as well. We've rejected essentially this same thing

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-01 Thread Martin Storsjö
On Thu, 1 Jul 2021, Jan Ekström wrote: On Thu, Jul 1, 2021 at 5:16 PM James Almer wrote: From: Matthieu Patou Suggested-by: ffm...@fb.com Signed-off-by: James Almer --- For the record, if Big Companies such as FB or Bilibili want to keep on dragging FLV/RTMP around I recommend: 1. Creat

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-01 Thread Jan Ekström
On Thu, Jul 1, 2021 at 5:16 PM James Almer wrote: > > From: Matthieu Patou > > Suggested-by: ffm...@fb.com > Signed-off-by: James Almer > --- For the record, if Big Companies such as FB or Bilibili want to keep on dragging FLV/RTMP around I recommend: 1. Create something like https://datatrack

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-01 Thread Derek Buitenhuis
On 7/1/2021 5:25 PM, Thomas Volkert wrote: > I was also about to prepare a discussion mail against this patch set. We > shouldn't fuzzy the spec. > > So.. +1 for skipping this patch set. Hard NAK from me as well. We've rejected essentially this same thing several times, and the reasonign has not

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-01 Thread RADSL
On 7/1/2021 9:23 AM, Jean-Baptiste Kempf wrote: On Thu, 1 Jul 2021, at 18:15, James Almer wrote: Is there an official source for these numbers? Apparently no. Neither does REALH263 or MPEG4 as far as i can see. But if there are files out there that can be demuxed with this change, as long as

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-01 Thread James Almer
On 7/1/2021 1:25 PM, Thomas Volkert wrote: On 01.07.2021 18:15, James Almer wrote: On 7/1/2021 11:41 AM, James Almer wrote: On 7/1/2021 11:27 AM, Andreas Rheinhardt wrote: James Almer: From: Matthieu Patou Suggested-by: ffm...@fb.com Signed-off-by: James Almer ---   libavformat/flv.h   

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-01 Thread Thomas Volkert
On 01.07.2021 18:15, James Almer wrote: > On 7/1/2021 11:41 AM, James Almer wrote: >> On 7/1/2021 11:27 AM, Andreas Rheinhardt wrote: >>> James Almer: From: Matthieu Patou Suggested-by: ffm...@fb.com Signed-off-by: James Almer ---   libavformat/flv.h    |  1 +

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-01 Thread Jean-Baptiste Kempf
On Thu, 1 Jul 2021, at 18:15, James Almer wrote: > >> Is there an official source for these numbers? > > > > Apparently no. Neither does REALH263 or MPEG4 as far as i can see. But > > if there are files out there that can be demuxed with this change, as > > long as we don't support creating them

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-01 Thread James Almer
On 7/1/2021 1:21 PM, RADSL wrote: On 7/1/2021 9:15 AM, James Almer wrote: On 7/1/2021 11:41 AM, James Almer wrote: On 7/1/2021 11:27 AM, Andreas Rheinhardt wrote: James Almer: From: Matthieu Patou Suggested-by: ffm...@fb.com Signed-off-by: James Almer ---   libavformat/flv.h    |  1 +   l

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-01 Thread RADSL
On 7/1/2021 9:15 AM, James Almer wrote: On 7/1/2021 11:41 AM, James Almer wrote: On 7/1/2021 11:27 AM, Andreas Rheinhardt wrote: James Almer: From: Matthieu Patou Suggested-by: ffm...@fb.com Signed-off-by: James Almer ---   libavformat/flv.h    |  1 +   libavformat/flvdec.c | 14 ++

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-01 Thread James Almer
On 7/1/2021 11:41 AM, James Almer wrote: On 7/1/2021 11:27 AM, Andreas Rheinhardt wrote: James Almer: From: Matthieu Patou Suggested-by: ffm...@fb.com Signed-off-by: James Almer ---   libavformat/flv.h    |  1 +   libavformat/flvdec.c | 14 --   2 files changed, 13 insertions(+),

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-01 Thread James Almer
On 7/1/2021 11:27 AM, Andreas Rheinhardt wrote: James Almer: From: Matthieu Patou Suggested-by: ffm...@fb.com Signed-off-by: James Almer --- libavformat/flv.h| 1 + libavformat/flvdec.c | 14 -- 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/libavformat/f

Re: [FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-01 Thread Andreas Rheinhardt
James Almer: > From: Matthieu Patou > > Suggested-by: ffm...@fb.com > Signed-off-by: James Almer > --- > libavformat/flv.h| 1 + > libavformat/flvdec.c | 14 -- > 2 files changed, 13 insertions(+), 2 deletions(-) > > diff --git a/libavformat/flv.h b/libavformat/flv.h > index 3

[FFmpeg-devel] [PATCH 1/3] avformat/flv: add support for h265 streams

2021-07-01 Thread James Almer
From: Matthieu Patou Suggested-by: ffm...@fb.com Signed-off-by: James Almer --- libavformat/flv.h| 1 + libavformat/flvdec.c | 14 -- 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/libavformat/flv.h b/libavformat/flv.h index 3571b90279..7a026d3217 100644 --- a/