From patchwork Mon Oct 4 20:52:20 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nicolas Gaullier X-Patchwork-Id: 30900 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a6b:6506:0:0:0:0:0 with SMTP id z6csp4694048iob; Mon, 4 Oct 2021 13:52:44 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwMaF1DD7HOfD2aGmfW8edCUtjrcVsACARzuZLgOeDsTZ+JMc13/ZRAM0uJCZQE0JlZDL1r X-Received: by 2002:a17:906:7ac4:: with SMTP id k4mr20485423ejo.430.1633380763856; Mon, 04 Oct 2021 13:52:43 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1633380763; cv=none; d=google.com; s=arc-20160816; b=HfjPX8AneU5wxfFCd+p5cXYnYXG6HzF6A4N/fZvG7wbrg308yr59oynB+b2ONYYCHU /TmyUiEZmf5fwzh+SnPHj4LNyrxCyUOxZTlg2LC/Kx9iHB3VIs7TyxxV5rmqbTefmTg6 IBRCMlIGRjAfU9VizmtxcjU0gSCdcxjN8HGGDgYqDKPouhYVkxFPgLt9lNK8YL/b9l+m SiNuM6IwQePVsFTRmgcUuos2hab1xVCz7VcuQfeAUdLpkoOn9IChAA/nG/5IrJBWfNWp cswladcmOk2JywEpIZtztxkO0auCn+gc53I7u5ZLIXvDyCDz0GJWEaalieyJAQf2MIj3 QcCQ== 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=5mHTB9sOw4GBJG1F3XLHtcPAfHwF68/8FqtelKj5MpQ=; b=PtSHzht37nzMsvzrgCs+QRTHwIrHLFbR/wPTwAcBy1FbSfTDDeB/pThXOYxW/civUr GahRwAkMkBY5plFLQ/sQudKg9wUa9p/Tks1iSVEq5Y/NeEhPS7AWaC3YdywJzcKCTy9Q nW0Kjhe6eoSgPDBGfeLNxpxW9xeQsQWtCPZU42+xhj3HjcA5hhv/niRZeECjnoU5F/bO lWwNUC+a6Xu2OCdjq3FEC8sNDxxH3W/hvU0nKeWoFwI9AxapIqNgdvfwUrVNV9be09zc SPXVWwNPEOhkcleT8saWWDOn2LU7SI9bFDZsiI/xaSWuGetyzF6bLRMxA2WAZiCQ2CSy vuPg== 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 f20si2221770ejk.645.2021.10.04.13.52.42; Mon, 04 Oct 2021 13:52:43 -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 2D32068A4AC; Mon, 4 Oct 2021 23:52:38 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from srv-infra-1.infra.inf.glb.tvvideoms.com (www.inf.tvvideoms.com [213.205.126.156]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 08C636881C2 for ; Mon, 4 Oct 2021 23:52:31 +0300 (EEST) Received: from cji.paris (unknown [172.16.3.159]) by srv-infra-1.infra.inf.glb.tvvideoms.com (Postfix) with ESMTP id 6CF252903A; Mon, 4 Oct 2021 20:52:28 +0000 (UTC) From: Nicolas Gaullier To: ffmpeg-devel@ffmpeg.org Date: Mon, 4 Oct 2021 22:52:20 +0200 Message-Id: <20211004205221.62939-1-nicolas.gaullier@cji.paris> X-Mailer: git-send-email 2.32.0 MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH 0/1] avformat: 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: EjJ/qlL6p6IE 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 expect some enriching discussion around this... thank you for your feedbacks... 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(-)