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 63A9C44C95 for ; Mon, 13 Feb 2023 18:10:03 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 07EFE68BE31; Mon, 13 Feb 2023 20:09:52 +0200 (EET) Received: from srv-infra-2.infra.inf.glb.tvvideoms.com (www.inf.tvvideoms.com [213.205.126.156]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id CF3B068BD2E for ; Mon, 13 Feb 2023 20:09:44 +0200 (EET) Received: from cji.paris (unknown [172.16.3.159]) by srv-infra-2.infra.inf.glb.tvvideoms.com (Postfix) with ESMTP id EE9DD42B54; Mon, 13 Feb 2023 18:09:43 +0000 (UTC) From: Nicolas Gaullier To: ffmpeg-devel@ffmpeg.org Date: Mon, 13 Feb 2023 19:09:30 +0100 Message-Id: <20230213180936.815-1-nicolas.gaullier@cji.paris> X-Mailer: git-send-email 2.39.1.windows.1 MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH 0/6] wavdev: s337m support 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 Cc: Nicolas Gaullier 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: This is the follow up of a past work https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=2593 In my understanding, the code review was running pretty well, but there was two issues: - the first one was fixed with the creation of the dolby_e parser which removed the need of one of the initial patches - the second one is much embarrassing for me, it is the need to be able to pass-through s337m (as in current code). This is typically required to remux s337m to mxf, as there is no s337m submuxer available yet. And the scope of this option is not clear to me. I have understood that by default, the s337m demux shall be enabled and I changed my code accordingly. I also understand we cannot multiply options for every little thing, so I proposed a global option: https://patchwork.ffmpeg.org/project/ffmpeg/patch/20211012154156.1726-2-nicolas.gaullier@cji.paris/ Indeed, it is interesting, for example when using ffprobe as a light "QC" tool and to reject for example mpegts files that do not have any single PMT, etc. But I had no answer despite several pings, so I suspect it is not the way to do. Here, I lastly propose to 'reuse' the AVOption used for s302m as the problem is really similar. At the end, I find this pretty cool like this... Hope you will like it too! Anyway, please give me a feedback. Sample file used for fate: https://0x0.st/zdW-.wav Nicolas Gaullier (6): avformat/s337m: Split read_packet/get_packet avformat/s337m: Consider container bit resolution avformat/s337m: New ff_s337m_probe() avformat/wavdec: s337m support avformat/wavdec.c: Reindent after last commit avformat/wavdec: Test s337m libavformat/s337m.c | 71 ++++++++++++++++++++++++++++++++++++---- libavformat/s337m.h | 54 ++++++++++++++++++++++++++++++ libavformat/wavdec.c | 61 +++++++++++++++++++++++++--------- tests/Makefile | 1 + tests/fate/audio.mak | 3 ++ tests/ref/fate/s337m-wav | 10 ++++++ 6 files changed, 177 insertions(+), 23 deletions(-) create mode 100644 libavformat/s337m.h create mode 100644 tests/ref/fate/s337m-wav -- 2.30.2 _______________________________________________ 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".