From patchwork Tue Oct 12 15:41:55 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nicolas Gaullier X-Patchwork-Id: 31086 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a05:6602:2084:0:0:0:0 with SMTP id a4csp4916711ioa; Tue, 12 Oct 2021 08:42:23 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwX2ATrdnOxt6EYnshZt/PKHLwCX39wZ7IY6rWoqSnjUw298VE/Ft4lO1eFFa6CRiYyTsvK X-Received: by 2002:a17:906:9a07:: with SMTP id ai7mr34186470ejc.55.1634053343517; Tue, 12 Oct 2021 08:42:23 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1634053343; cv=none; d=google.com; s=arc-20160816; b=octNg3961dJjPsPYLuNs69VN5vvf8V8OPGU4jKTh/2T2OOK2Gs8KJ0zKzAISBgagRH /f3oESDl/Ho5NvA6gvEvj/q7OGNjvliGkbLwDyQ4CgU2vwKwKm66Oa9dkA5Z6T6q1g/s 1+AcGcOU/wtCM+HK4heQo5vCbbXxlLbFT8PdaLoeVH2WSOtSfkjWk0E1yY90zymJd86t 8Z7onMAcizrLZRkgq4Mv4VDUHWxyiNdYm7fWO28eo/AzBPTfbAihMTNCYBSzKlVe0Ef1 PX2bwTqYwKTDxrSo0Vu9KPpOLzGLuywd3HTFGOZ8Py+zaKRuzB5e2sNzMfOI8gsxES55 TYiw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:cc:reply-to :list-subscribe:list-help:list-post:list-archive:list-unsubscribe :list-id:precedence:subject:mime-version:message-id:date:to:from :delivered-to; bh=tINg/nbsGhETL9mS/SETbdFMGSBdOxTBTDv1PBS0RsE=; b=gb2SrQug1T6Pld+6thz8xN4hIoBwDYhsgdBBb6XxxFa/SLexjsYnfrBnW9fymVH+if xbOKpvbAHhFqrc9jcXqkPkkaTS85IbjPY5Gc3eIUAcQutIWkpBmDXqMkIg4e/99jYC6Z LxxwZ7mtI7dP1RLfrb3nuPwb3Ub49ATLoQ9Ziq4jkb09xaPDAWAgp6BQ57CID2eFAJjD zWNanPhC+CtSU0sqQs78cP8O1ArlmKEeKveN85jq5eByNYSMQYzpYm965q7zKHPKuuPb Mi/FBkaP0QpWozcXv5eEaW6Z1kvpLO96dOHNlGf5/NHSb2TyJaLNU/7W9QWUJdYNOQDz uf9A== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id lf1si16731319ejb.498.2021.10.12.08.42.23; Tue, 12 Oct 2021 08:42:23 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id AD84268A75A; Tue, 12 Oct 2021 18:42:06 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org 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 2C703689ADA for ; Tue, 12 Oct 2021 18:41:59 +0300 (EEST) Received: from cji.paris (unknown [172.16.3.159]) by srv-infra-2.infra.inf.glb.tvvideoms.com (Postfix) with ESMTP id 2625A418CD; Tue, 12 Oct 2021 15:41:58 +0000 (UTC) From: Nicolas Gaullier To: ffmpeg-devel@ffmpeg.org Date: Tue, 12 Oct 2021 17:41:55 +0200 Message-Id: <20211012154156.1726-1-nicolas.gaullier@cji.paris> X-Mailer: git-send-email 2.33.0.windows.2 MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH v3 0/1] Add-probe_streams-option 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 Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: bkVvR98r3Nnj >This is a following to a precedent work which was a too long patch serie: >https://patchwork.ffmpeg.org/project/ffmpeg/list/?series=2593 >At the end, s337m will be probed (if the user choose to) in WAV files. >But it is important to maintain workfows where typically WAV files are remuxed in MXF with s337m going through. > >I have reworked on what I think a user might expect when disabling codec auto-detection. >I have not disabled stream probing for some very specific or legacy cases (asf, avi, mpg/Hikvision,sofdec) but maybe I should have. >The main idea is to always trust the descriptors, even if it means crashing. >Typically, an mpegts file having no PAT/PMT will not be supported at all. There is a clear use case for that, for example when using ffprobe in front of a transcoder that is not ffmpeg and not able to handle such files. I am surprised there was no reaction to this patch: there is for sure things to discuss about what is a proposal for now. This patch is required for me before rolling out my patch serie for s337m support in WAV files. (This v3 is just a rebased version) Thank you! Nicolas Nicolas Gaullier (1): avformat: Add probe_streams option doc/formats.texi | 3 +++ libavformat/avformat.h | 9 +++++++++ libavformat/mpeg.c | 5 +++-- libavformat/mpegts.c | 10 ++++++---- libavformat/options_table.h | 1 + libavformat/version.h | 2 +- libavformat/wavdec.c | 8 +++++--- 7 files changed, 28 insertions(+), 10 deletions(-)