From patchwork Mon Jan 11 17:37:02 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nicolas Gaullier X-Patchwork-Id: 24914 Return-Path: X-Original-To: patchwork@ffaux-bg.ffmpeg.org Delivered-To: patchwork@ffaux-bg.ffmpeg.org Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by ffaux.localdomain (Postfix) with ESMTP id 1735F44B8FC for ; Mon, 11 Jan 2021 19:37:14 +0200 (EET) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id EBF6868ABC7; Mon, 11 Jan 2021 19:37:13 +0200 (EET) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from srv-infra-1.infra.inf.glb.tvvideoms.com (unknown [213.205.126.156]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id E47E868ABFE for ; Mon, 11 Jan 2021 19:37:06 +0200 (EET) Received: from cji.paris (unknown [172.16.3.159]) by srv-infra-1.infra.inf.glb.tvvideoms.com (Postfix) with ESMTP id EC8EA1FFED; Mon, 11 Jan 2021 17:37:04 +0000 (UTC) From: Nicolas Gaullier To: ffmpeg-devel@ffmpeg.org Date: Mon, 11 Jan 2021 18:37:02 +0100 Message-Id: <20210111173703.1736-1-nicolas.gaullier@cji.paris> X-Mailer: git-send-email 2.27.0.windows.1 MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH 0/1] avformat/dolby_e: Add a parser X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.20 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 Cc: Nicolas Gaullier Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Note: my first intend here is, afterwards, to provide s377m support to the wav demuxer, (the actual code leads to pts issues : I sent a workaround https://patchwork.ffmpeg.org/project/ffmpeg/patch/20201215171324.99-1-nicolas.gaullier@cji.paris/ that was rejected, so this new work for a Dolby E parser is simply a follow up). I have tested that this patch actually fixes PTS generation when demuxing s337m wav samples. This is a first try, so, yes, I kown it is missing Changelog and version updates. I think the review will take some time so this could be updated lastly. This is a single patch, I think it is easier for review anyway, but I expect you will ask me to split it in several files like - dolby_e_parser.c/.h - dolby_edec.c/.h Anyway, if so, I think I would keep this first patch as is and simply add a new patch to just split all the things. Unless you think it is better to split the parser code from the very beginning/first patch. So, I keep listening about how this work should be presented, in term of number of patches etc. Thank you in advance for the review Nicolas Nicolas Gaullier (1): avformat/dolby_e: Add a parser libavcodec/dolby_e.c | 260 ++++++++++++++++++++++++++++--------------- libavcodec/dolby_e.h | 38 ++++++- libavcodec/parsers.c | 1 + 3 files changed, 202 insertions(+), 97 deletions(-)