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 164A04B1DE for ; Mon, 1 Jul 2024 20:56:22 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 807E468D865; Mon, 1 Jul 2024 23:56:20 +0300 (EEST) Received: from mail-ed1-f42.google.com (mail-ed1-f42.google.com [209.85.208.42]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 47AE268D65F for ; Mon, 1 Jul 2024 23:56:14 +0300 (EEST) Received: by mail-ed1-f42.google.com with SMTP id 4fb4d7f45d1cf-57d10354955so1036396a12.1 for ; Mon, 01 Jul 2024 13:56:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1719867373; x=1720472173; darn=ffmpeg.org; h=user-agent:in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:to:from:date:from:to:cc:subject :date:message-id:reply-to; bh=aLIewCet6pUVlqwB5GQxLWt5I9X4lGlhyJC+vN/uRK0=; b=W9v6JwIzi1Uuz/sc3OHkn8ANFFW/mOF67YC7aDcU0sth2i2GYKZG/6aEPvBxEpRvX7 WU263KUEMKevfA07HdcizQL6OwtE/pdiPYQ+bWnAi53pSB+ETYB6+7AJ7bto8cWK8amo 2CSVL2P7eYP48wirI7TDxOVuAbUrq5SHxoUTxJUb7QN/zDI0XITKLSbmbO4V0ueK9/gC QHHdGY0pAl0phrIAIzzbxonVRjfBh8tRKlBki+2Byw3I6s1+YRaJPtZX/HzG2wNMS+bL 6MFR29TaKJrpoleRWsx19K9AHRZdmZKmrzoij7riyFODs1DoS8cEWFXqSs00/I2Ms363 sq1A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1719867373; x=1720472173; h=user-agent:in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:to:from:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=aLIewCet6pUVlqwB5GQxLWt5I9X4lGlhyJC+vN/uRK0=; b=AUgR7/FuzMk8xtxjC9S0woHB885hbiYuOOgdFcdLV1H//1I+uwrQ6NzXdvHF0CJy8q GSjMltbwXdV4CL+K5UQq9f7SE7Ts5kjX8tLwHveNeicBhNTN/wtwVTUHz5GGBbS2dRQb H/ubjJ9HP0O9LEsXb4F2+6H3pcTTjKXCI4XF5eFIEqCEgKfqYjVvAofM6I6OZCsdl78W tF/PSxq8CDCZaVQWdPhFK5QExkHCTFNAgqnb1SybfS0kxhyeXkSiia3qMVfAo5KhBDus fflJyHQx/3z26/I1E6/Hu09uIMfKOlgHLv7Tn1k4ci3e6uP2q5GLOko4NU9kdXsvZadl O+xQ== X-Gm-Message-State: AOJu0YzssqX71jn6LAZb3sx+b9793fblFEji6q2COC/rwo0bgACYe6Zz xOCpDzzP5fUTW4CVfGVyFHBYe0MLK9NtU0fBxZilHa+uV2JUuczROBOp6Lec5F4= X-Google-Smtp-Source: AGHT+IFrB/1mIH/R0480eqDl831DHNsYpHIvNfsZaTUcZYa+pj2134DqjwylUywquSO+zS905zKs8Q== X-Received: by 2002:a05:6402:2547:b0:57d:f9f:133b with SMTP id 4fb4d7f45d1cf-5879f0c4da6mr4461716a12.2.1719867372775; Mon, 01 Jul 2024 13:56:12 -0700 (PDT) Received: from mariano (dynamic-adsl-84-220-189-10.clienti.tiscali.it. [84.220.189.10]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-5861381756esm4856333a12.56.2024.07.01.13.56.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 01 Jul 2024 13:56:12 -0700 (PDT) Received: by mariano (Postfix, from userid 1000) id 83194BFCE8; Mon, 1 Jul 2024 22:56:10 +0200 (CEST) Date: Mon, 1 Jul 2024 22:56:10 +0200 From: Stefano Sabatini To: FFmpeg development discussions and patches , Marth64 Message-ID: Mail-Followup-To: FFmpeg development discussions and patches , Marth64 References: <20240627030546.143368-1-marth64@proxyid.net> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/2.1.4 (2021-12-11) Subject: Re: [FFmpeg-devel] [PATCH] avformat/dvdvideodec: Fix duration logic with 1 chapter and validate chapter range 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 date Monday 2024-07-01 22:49:42 +0200, Stefano Sabatini wrote: > On date Wednesday 2024-06-26 22:05:46 -0500, Marth64 wrote: > > Chapters and duration are calculated together in dvdvideo demuxer. > > Previous chapter calculation logic treated extraction of 1 chapter > > using chapter_start and chapter_end switches incorrectly, returning > > the duration of the entire title instead of just the segment. > > > > > Fix the logic so that it calculates and returns the duration of the > > chapter segment instead. Additionally, validate that chapter_end > > exceeds chapter_start (except in the special case of 0). > > What is the meaning of the 0 special case? > > > > > Signed-off-by: Marth64 > > --- > > libavformat/dvdvideodec.c | 15 ++++++++++++--- > > 1 file changed, 12 insertions(+), 3 deletions(-) > > [...] > > Looks good to me. Both patches applied. _______________________________________________ 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".