From patchwork Thu Apr 8 23:20:02 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 26821 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 C1E58449C88 for ; Fri, 9 Apr 2021 02:20:35 +0300 (EEST) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 9177268047E; Fri, 9 Apr 2021 02:20:35 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-oln040092075022.outbound.protection.outlook.com [40.92.75.22]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 1D68C68004A for ; Fri, 9 Apr 2021 02:20:28 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=lZhOg15RuxFcS3HG+mW5zLE4nx2GP4fOxDIbPamjUeyNVgYA9pYPkNwOe4uogX8/+66eaV07GQAvzz8mPaelvJZXhc7SaJRfF1AUNjFv/93zAKQJe/ZRxhm+tP6aLbFNMtvg0UyVaRoFVT7Dg905VAqx/wvp6l6l37QDi6s/c0dNABPsYUK87dijqJNkM2nwBWC66p+ohWF1xBWh62D+CaaP2ekwpMVV6ocKhvAUVrhxNYP15lJAuLAviSJyS37JaX1pqYixpqihivJ/p8dasi3CybF45VLRjpuVSeFZkZaDBBhSW2f5uzsm4FdY3J9qZFXxTaJGkPu66TFvc9KNRA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Kv5mkDFOR0yq8GjlMeY4/a6MyAEaIlPFcl3w2K2J/hM=; b=NBcOvaRLfUoflJ25dmNJSyCO2hM/qu0CLOvJz+yPpu8kmxJTEmVbCzZZdzGof7TXwilhfWrYt6/KFMeIrz7Ptya5O0hklbaAJAVp0eaEa+mj0a1cf7fPFG+Cv0Te6jHJhcmFpu3Fg8I7rw5sjtkS048RZ9p1yNbTV4Rg1+Nn6hvP1x7Qms7LIp8A+P6xKsTQBXHBz63yJ95+OEY41pndPyCaPPY65yQ50FG8z60w+5Jx5F77BtjpOx/s4Pu56bsHmWxVYQZwLbzFKmgLM8EDm3bS7AQInjEjXOAo8GNwdJJ9k0cqOWUCmp4AcV3FMPEUbS4+iAPkwUTDweIACEhTDg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Kv5mkDFOR0yq8GjlMeY4/a6MyAEaIlPFcl3w2K2J/hM=; b=YaVmG0JjsV3SBaQQGSjaptAzMknqe+BrxjCMDKMa92sVZO03wIeUYphXAkNqFjOlfTwAWAXpA83vVWXbbnkIPTTFFxRx56O1BYf4FMIXrQdgFY21PMuipIWrUN94dELdikb5HzAC6oUnkc4Rr0TwNt05UCbuG9XGPk0UE9TlVZKGUX329TCjWoh1ckqlezKhU3HySFuURG7jZgofPibH+DsIy8VzgD5THu0QAfV9T1gFGoXdPb8HCKebToNI2CbQi/T8hhSDcZC0vrL4Z545u9Sm9otg0imYlQBbF0yE9GSUtJHrQICRUkzyevpvilakYod2W7EDrtoSNpR8Uzle6g== Received: from HE1EUR04FT036.eop-eur04.prod.protection.outlook.com (2a01:111:e400:7e0d::41) by HE1EUR04HT155.eop-eur04.prod.protection.outlook.com (2a01:111:e400:7e0d::184) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4020.17; Thu, 8 Apr 2021 23:20:27 +0000 Received: from HE1PR0301MB2154.eurprd03.prod.outlook.com (2a01:111:e400:7e0d::52) by HE1EUR04FT036.mail.protection.outlook.com (2a01:111:e400:7e0d::148) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4020.17 via Frontend Transport; Thu, 8 Apr 2021 23:20:27 +0000 X-IncomingTopHeaderMarker: OriginalChecksum:01351AC9BC3A3C8F363965A61B251440F62F30079728422A80C27DC2EA7C893E; UpperCasedChecksum:65294C974C9F24052646BE659785C9E4A1EDB598E57F87DF5D24BBF90FB640E4; SizeAsReceived:7370; Count:45 Received: from HE1PR0301MB2154.eurprd03.prod.outlook.com ([fe80::8128:5de5:4e94:9a21]) by HE1PR0301MB2154.eurprd03.prod.outlook.com ([fe80::8128:5de5:4e94:9a21%3]) with mapi id 15.20.4020.018; Thu, 8 Apr 2021 23:20:27 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Fri, 9 Apr 2021 01:20:02 +0200 Message-ID: X-Mailer: git-send-email 2.27.0 X-TMN: [ul7LDM2HDuIVjP5lIofF1mGb4u1NJtCS] X-ClientProxiedBy: FR2P281CA0004.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:a::14) To HE1PR0301MB2154.eurprd03.prod.outlook.com (2603:10a6:3:2a::22) X-Microsoft-Original-Message-ID: <20210408232003.1282946-1-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from sblaptop.fritz.box (188.192.137.96) by FR2P281CA0004.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:a::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4042.6 via Frontend Transport; Thu, 8 Apr 2021 23:20:26 +0000 X-MS-PublicTrafficType: Email X-IncomingHeaderCount: 45 X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-Correlation-Id: b8457149-4a95-449d-0d88-08d8fae4e491 X-MS-Exchange-SLBlob-MailProps: S/btQ8cKWiQDZtlYU9il0kuGVxzoXM2vrkF/ASGPmxeKEfdYgjjjibpb6B1iANaudpKYx5U+qCSVKMTBjdayAUELKrsHdRq5W95WZiAOzCUjQXfYRtMx0jf64FZolxxxG7XkYGO88A6XECRPmGHRfCRcFctuViiERx4EFGfjnVb2NPu/UQiCkvqAVtAaJSACd1S3RDwviEFu0vspRu1GZnXJRZRmaTp/Qoc4NznskKwOc475vAP97V5uEZ/h4+GpPBCwCfKG6D9IvQxLXGw4m41yT3ClHbqc9L6x6cMwjjE6DwAM9j2BcFndaQ7Qdj7NmZZERC+BQLL9OK77ptB4+NGnOOGAgFKmLI7Zy27i7acj+bUoFvzVE7gj5zwlPz4fP0IUMF0KwWMvwl6FYKS5HOVLJ1rVmQrb8f55e+KGSV4tBwfyjkeaeoXo8KRAf7oBY2kF0fQ0YNHjz1nf85Pb3gA3+lISjqD1NiEnkecaOjwrMNCE3hCYOK1NdYvdlMR0dnTUGU4s0ghQzC/aTtypYpHBD9s1Ja8kTynfhBjPvMq6wiEqUOE1ZWx0m1zJbnzs56ZpnEc3ANxYC7fXkAfEDAjQCr5TCTfSkE6xi6lkHYG8HiOn5gD7/4e4y1rnTW8bDqQTVzXmZU/VkkB2u6V/WJ/mVOlGu5lINYKgj0QVZtT8iUmrVC6bQYwA6Wu4Z69c0DR97zk15sCEI0lTDHaMGMRQhgVXUEM+b9txq2vZ0DenhLERoeC61XUGHljrYKzU5I/bzncfRas= X-MS-TrafficTypeDiagnostic: HE1EUR04HT155: X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: cN/ZSZjn5DojqIyEzrNwPknNkIBm/M6h3gJqY3/a9lwrlZR17zZupCq1LYA4+j9OmdYZTxeAmC9iKATX++JZxJ24Ze5RiFXT8eOyq3gfyZeeUsiefwzD0xZM0Xv5T31zPVehhSNn47OXoG5BJSXKK/oCuOJ/6LUS/VFwGjxupGqfIs15CqSUwmvF0jTPYq3DoS2vw0giy2lbssBaqwr/qTeA5uVEA6GYfizwe6HFrVCDL+Ko3/X0ehoVwRaVGSKZ1894pEPbImhb1QpG/AC+MeWnCmZYBLa/shR3YFzN3Pnsl9vTSBShJEMdlgApMP2NhqvisQ0kvkYC8qsUPkEJyfrFgaME/nJIbox2A3/pZIckscHuh1Z/vH/EwgsUZibjPiqKQ7DQ64Cz3dMbLY36pA== X-MS-Exchange-AntiSpam-MessageData: 8q3w0N5u2yFQwcYkxcZLzqGdnPfIsDXJ26oDiwZxhHQswRlTj6J/SYYQ9uPamcNK+ci8JxAdXsS5XrUb6O335k9ubrjYeGhm0PLdaFXhabh5A1zFDI2vTzrkyeuKXhiz/yam2X7tIb4Rt13Bw9YIyg== X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: b8457149-4a95-449d-0d88-08d8fae4e491 X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Apr 2021 23:20:27.2561 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-AuthSource: HE1EUR04FT036.eop-eur04.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: Internet X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1EUR04HT155 Subject: [FFmpeg-devel] [PATCH 1/2] avformat/matroskaenc: Remove remnant of inline-timing subtitle packets 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: Andreas Rheinhardt Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Signed-off-by: Andreas Rheinhardt --- libavformat/matroskaenc.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index 609a588f78..a4ddf1fa6a 100644 --- a/libavformat/matroskaenc.c +++ b/libavformat/matroskaenc.c @@ -2187,7 +2187,7 @@ static int mkv_write_vtt_blocks(AVFormatContext *s, AVIOContext *pb, const AVPac put_ebml_uint(pb, MATROSKA_ID_BLOCKDURATION, pkt->duration); end_ebml_master(pb, blockgroup); - return pkt->duration; + return 0; } static int mkv_end_cluster(AVFormatContext *s) @@ -2358,9 +2358,9 @@ static int mkv_write_packet_internal(AVFormatContext *s, const AVPacket *pkt) } } else { if (par->codec_id == AV_CODEC_ID_WEBVTT) { - duration = mkv_write_vtt_blocks(s, pb, pkt); - if (duration < 0) - return duration; + ret = mkv_write_vtt_blocks(s, pb, pkt); + if (ret < 0) + return ret; } else { ebml_master blockgroup = start_ebml_master(pb, MATROSKA_ID_BLOCKGROUP, mkv_blockgroup_size(pkt->size, From patchwork Thu Apr 8 23:29:47 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Andreas Rheinhardt X-Patchwork-Id: 26822 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 D00F644A393 for ; Fri, 9 Apr 2021 02:30:09 +0300 (EEST) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id A8F27689972; Fri, 9 Apr 2021 02:30:09 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from EUR06-VI1-obe.outbound.protection.outlook.com (mail-vi1eur06olkn2019.outbound.protection.outlook.com [40.92.17.19]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 25AAB689972 for ; Fri, 9 Apr 2021 02:30:03 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=j3v5K5TWYoQq07fE1FH0hwosjenJQYU4UppHIUpb9+3Y5oe6Jt+O0k/Pj+NU/PZuQ9flA//FuB3kunzGJUL2vQTcRJC04wLJLfo1j0OVNyKD6cV5K1K43pvHOq5yZbR+ZtqbDz2AtPEiFABWMMDd44/TxCiYmIY2+0iDMl051iIOXFl9/bTH1faPKjkyAR06MM4dZVn2OLHtPEph48J/5JzHDChK5rYFZKBpez3tWMcQuqw7UdcPuGB8ILgWi4lSSh47XkFyGCZ6lgSTNcIeZaLtIAnQrBwtO4ZCRCQ/6TMjLeYOETHQ8mzeex6oYarWJLeXn5cYqbO6ZOupGQIyAQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=tu6BjwN3ULxzDIYA4ADjTrIp7n8ZQorTHNj2q2iUxA8=; b=CzRzItQG/dfgOQVj7rnlxE6Ij+w44aFAjMbQ2h2yEXsJV8c7qL0EwRDWK/TYRq/q6FxIe1M5KKmrclkMLEcP/IfRoPupPrdwqOFjJVsAnfNSbyJgdM89R3Q4h7QKVdOkjOrKm3NJWRjoYGabukZLN1C/7FU18GuPNyMRcId/IG7f2G903DVe/PA2JZMq89MvVwAlo48xDiqb6m1kzQip8JNuVHeG5i2M1Hz4Ktblw6hm+GTgGzt2yynL+2WT4H8gcieAosSQHETixVlLWYJVF/k/xASzYfLentzlTBjonq5FG3sguJLqJ4jwyjLQMHXvWZaS89z+Y/tLl4/rYiJvFA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=tu6BjwN3ULxzDIYA4ADjTrIp7n8ZQorTHNj2q2iUxA8=; b=TwbnXK2vO0J4hDDEWNVJIO/pP/WZ5EXoxZ9OBJYGLfLMdhWoekzgzzl6Tlxif5/QMCDtny9P5WemOdulxgFopBqPoyEG5d+x8eFcunwfp7hECO5BMzl6ObD9vQY6uEu5Up5KBP4/yfh8qDAMirNcVPEljDxZohZQXAwyFpExHX23CoKVfUZIVYYqhndbtwbcwu4Msy8MqZFdC+K5NmI/ZyiP7EtJHjosxqzt819WwK11uVEVt82mm0C6p3iIRg2pgg2y0wtXKRKWjPxduKxoNdq1WQEO6n2xRCnf7eaET0D3Nzs5DJEnR3QrBsHv6U4/ZeBpuMWweshIwf7iXvBhuw== Received: from DB8EUR06FT015.eop-eur06.prod.protection.outlook.com (2a01:111:e400:fc35::4f) by DB8EUR06HT243.eop-eur06.prod.protection.outlook.com (2a01:111:e400:fc35::284) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4020.17; Thu, 8 Apr 2021 23:30:02 +0000 Received: from HE1PR0301MB2154.eurprd03.prod.outlook.com (2a01:111:e400:fc35::4d) by DB8EUR06FT015.mail.protection.outlook.com (2a01:111:e400:fc35::156) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4020.17 via Frontend Transport; Thu, 8 Apr 2021 23:30:02 +0000 X-IncomingTopHeaderMarker: OriginalChecksum:3791692F448610D4BBE0B17FF314013887DB6D5D2A88931F21C16A5980804AE4; UpperCasedChecksum:15A855F60F7E7652BA9E5678C341489F1F18EADE4C970F948E5E9862A7AA2C66; SizeAsReceived:7560; Count:47 Received: from HE1PR0301MB2154.eurprd03.prod.outlook.com ([fe80::8128:5de5:4e94:9a21]) by HE1PR0301MB2154.eurprd03.prod.outlook.com ([fe80::8128:5de5:4e94:9a21%3]) with mapi id 15.20.4020.018; Thu, 8 Apr 2021 23:29:55 +0000 From: Andreas Rheinhardt To: ffmpeg-devel@ffmpeg.org Date: Fri, 9 Apr 2021 01:29:47 +0200 Message-ID: X-Mailer: git-send-email 2.27.0 In-Reply-To: References: X-TMN: [5S4I48WHYXsJwhMQc6m6F1tW9wmod2W7] X-ClientProxiedBy: AM0PR03CA0056.eurprd03.prod.outlook.com (2603:10a6:208::33) To HE1PR0301MB2154.eurprd03.prod.outlook.com (2603:10a6:3:2a::22) X-Microsoft-Original-Message-ID: <20210408232947.1333713-1-andreas.rheinhardt@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from sblaptop.fritz.box (188.192.137.96) by AM0PR03CA0056.eurprd03.prod.outlook.com (2603:10a6:208::33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4020.16 via Frontend Transport; Thu, 8 Apr 2021 23:29:55 +0000 X-MS-PublicTrafficType: Email X-IncomingHeaderCount: 47 X-EOPAttributedMessage: 0 X-MS-Office365-Filtering-Correlation-Id: 4e079225-6398-4b3c-6f61-08d8fae6377e X-MS-Exchange-SLBlob-MailProps: +LiGfBxqLEvTZdiKx5OEHV33ImH+9b3M1/DahkVefHk4AifvvUGi7ED89YbmMsrDm+vETaEy74BRMXyJaiI6GH1yGJZ1oVNWwbpHTyFsZ8lReqOhTAa9dUNluzh+wHduKXmrVa3Przg4zWdqaXK+O5MhICY6LpNCaMLz3URv7o9YM7ki54RBJ/2+VPEMunpEs/H0DIqq8/0w4HVs5gu2ylxibcm2YvAfZa4mQlDzO7Hc2juzE5IVr5mp2pDWFYv7iAvIOF3mUufPhkWpnYHlto/vN/UN8sOZ7pO/FjjX6+S4mOtSFKzB5SxiOizF/URryf5lUdBzkKAbQMZL9YT8Vds2AxQ4znUhbI2njFMSsvvh/+Co6i/Z8mjPzkXQ5S0FJkymT4DP3h8qI2KPAyvxQebf1F0jcUjHkhWTREiWTztXdV01KWsc7l9VdmkRCve3/ea2gK2UY3p/gP15IUxFHAbVeEe0LEqTWV3PzQQW2j389Zd7sbREL899llOdbzE5KHOf/tHGIVwTYtu1eA0NGZF12MQZmxmPfj3JIDmaylvHiAyuBZAnENYaVSwWdDlk4HLDvm6p6h0qJECtlgsx4bGxaylby6IG/nfS+gwhumGBiPw1dqV+1rsqUOs1VjxxG/Wm/wTMnQ1LSSSyk/FA5cB0iMGcnSFIO/nnayDtxYwf0mDEIIdxr0yo17uvPkzD3zCElUhzATWtyZh827VIUc/opzyuMi2RFQqbRTBJh0WoZ2RHpSajqEqdCDv+jGQYXJYXwxZq88v+hAdPgwG/q+IsIs3MbKr7 X-MS-TrafficTypeDiagnostic: DB8EUR06HT243: X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 1brbmY7vLdICLXi7vq6QqLPrPn2j/4Cy6lpgsYSzBtvJ43cJ/HvVjmjxo2I9xt4aRHL81w688VnE6cRkJRKzmn82yEmOT8r6rMZeSG1bukIovbSSrPMjV96y+p5GJ6cTMaoLk2tOTCfyV+CDSdAsG5ee2vo2+3CPHpckYb5nZ2eCTJSEtSrojHz8/g6xhkirCqdcoHsLQIudXQkNGJJanad9buNjdJCqxP7GFnjbDddurhrkalL1KC11PPV+1aP6qRV5tbfTL5LjPSEGgNrQsQ0znMVZK/TONnU+lAqZChVKWshjQaJed1LpyzUYBhD/Q7kjE7OJxB1bWBeS0LTQip/3IVd/tRRPNo/hraMQauaRoiU9wDdtk4VB8Z9g4VyY X-MS-Exchange-AntiSpam-MessageData: LULazPocluZwXjRwbsfS/X3jfUT4GIcEtsVE6P/8KAfYy3dZTF0ufjUzbb9KjYYG6OzDE1xlL6LkjzMat0bYYQsIvmETMrkWyZvhG/fwkVepzfAzGWufvpBkSvxthXwqqHBIlgfWaeB/Vk5TNJB1DA== X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4e079225-6398-4b3c-6f61-08d8fae6377e X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Apr 2021 23:29:55.5869 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-AuthSource: DB8EUR06FT015.eop-eur06.prod.protection.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: Internet X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB8EUR06HT243 Subject: [FFmpeg-devel] [PATCH 2/2] avformat/matroskaenc: Put subtitles without duration into SimpleBlocks 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: Andreas Rheinhardt Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" The value zero for AVPacket.duration means that the duration is unknown, which in practice means "play this subtitle until overridden by the next subtitle". Yet for Matroska a BlockGroup with duration zero means that the subtitle really has a duration zero. "Display until overridden" is achieved by not setting a duration on the container level at all and this is achieved by using a SimpleBlock or a BlockGroup without duration. This commit implements this. Signed-off-by: Andreas Rheinhardt --- libavformat/matroskaenc.c | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index a4ddf1fa6a..725e44c962 100644 --- a/libavformat/matroskaenc.c +++ b/libavformat/matroskaenc.c @@ -585,7 +585,7 @@ static int mkv_assemble_cues(AVStream **streams, AVIOContext *dyn_cp, put_ebml_uint(cuepoint, MATROSKA_ID_CUETRACK , tracks[idx].track_num); put_ebml_uint(cuepoint, MATROSKA_ID_CUECLUSTERPOSITION , entry->cluster_pos); put_ebml_uint(cuepoint, MATROSKA_ID_CUERELATIVEPOSITION, entry->relative_pos); - if (entry->duration != -1) + if (entry->duration > 0) put_ebml_uint(cuepoint, MATROSKA_ID_CUEDURATION , entry->duration); end_ebml_master(cuepoint, track_positions); } while (++entry < end && entry->pts == pts); @@ -2307,7 +2307,7 @@ static int mkv_write_packet_internal(AVFormatContext *s, const AVPacket *pkt) AVCodecParameters *par = s->streams[pkt->stream_index]->codecpar; mkv_track *track = &mkv->tracks[pkt->stream_index]; int keyframe = !!(pkt->flags & AV_PKT_FLAG_KEY); - int duration = pkt->duration; + int64_t duration = pkt->duration; int ret; int64_t ts = track->write_dts ? pkt->dts : pkt->pts; int64_t relative_packet_pos; @@ -2344,14 +2344,17 @@ static int mkv_write_packet_internal(AVFormatContext *s, const AVPacket *pkt) relative_packet_pos = avio_tell(pb); - if (par->codec_type != AVMEDIA_TYPE_SUBTITLE) { + if (par->codec_type != AVMEDIA_TYPE_SUBTITLE || + (par->codec_id != AV_CODEC_ID_WEBVTT && duration <= 0)) { ret = mkv_write_block(s, pb, MATROSKA_ID_SIMPLEBLOCK, pkt, keyframe); if (ret < 0) return ret; if (keyframe && IS_SEEKABLE(s->pb, mkv) && - (par->codec_type == AVMEDIA_TYPE_VIDEO || !mkv->have_video && !track->has_cue)) { + (par->codec_type == AVMEDIA_TYPE_VIDEO || + par->codec_type == AVMEDIA_TYPE_SUBTITLE || + !mkv->have_video && !track->has_cue)) { ret = mkv_add_cuepoint(mkv, pkt->stream_index, ts, - mkv->cluster_pos, relative_packet_pos, -1); + mkv->cluster_pos, relative_packet_pos, 0); if (ret < 0) return ret; track->has_cue = 1;