From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 5BC8D4B0CB for ; Tue, 28 May 2024 15:15:10 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id D691768D44A; Tue, 28 May 2024 18:15:08 +0300 (EEST) Received: from b-painless.mh.aa.net.uk (unknown [81.187.30.52]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 426EA68D2A5 for ; Tue, 28 May 2024 18:15:02 +0300 (EEST) Received: from 0.e.f.0.c.8.2.e.1.0.c.4.0.c.3.6.0.5.8.0.9.1.8.0.0.b.8.0.1.0.0.2.ip6.arpa ([2001:8b0:819:850:63c0:4c01:e28c:fe0] helo=andrews-2024-laptop.sayers) by painless-b.tch.aa.net.uk with smtp (Exim 4.96) (envelope-from ) id 1sByXV-007ANX-1W for ffmpeg-devel@ffmpeg.org; Tue, 28 May 2024 16:15:01 +0100 Date: Tue, 28 May 2024 16:14:59 +0100 From: Andrew Sayers To: FFmpeg development discussions and patches Message-ID: References: <20240521090316.782-1-timo@rothenpieler.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20240521090316.782-1-timo@rothenpieler.org> Subject: Re: [FFmpeg-devel] [RFC 00/13] flvdec/flvenc: add support for enhanced rtmp codecs and multitrack/multichannel X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On Tue, May 21, 2024 at 11:02:09AM +0200, Timo Rothenpieler wrote: > This is based on the preliminary spec for enhanced rtmp v2: > https://veovera.org/docs/enhanced/enhanced-rtmp-v2 > > The spec is not final, and can still undergo breaking changes, hence this set is purely for comments and review, and not ready to be merged until the final v2 spec is published. > > There are no samples out in the wild yet, so testing interoperability with other software has not happened yet either. > Specially the two other multitrack modes, where multiple tracks are in the same packet, have not been tested at all, since no software can write such files. > > The set can also be found on GitHub, where ignoring whitespaces makes specially the last patch a lot more readable: > https://github.com/BtbN/FFmpeg/tree/enhanced-flv > I ran this against a little review bot I'm working on. Please do s/\* / \*/g on the following: avformat/flvenc: add support for writing multi track audio > +static void flv_write_multichannel_header(AVFormatContext* s, AVCodecParameters* par, int64_t ts, int stream_index) avformat/flvenc: write enhanced rtmp multichannel info for audio with more than two channels > +static void flv_write_multichannel_body(AVFormatContext* s, AVCodecParameters* par) > +static int flv_get_multichannel_body_size(AVCodecParameters* par) > +static void flv_write_multichannel_header(AVFormatContext* s, AVCodecParameters* par, int64_t ts) avformat/flvenc: add enhanced audio codecs > +static void flv_write_aac_header(AVFormatContext* s, AVCodecParameters* par) avformat/flvenc: Implement support for multi-track video > +static void flv_write_codec_header(AVFormatContext* s, AVCodecParameters* par, int64_t ts, int stream_index) { _______________________________________________ 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".