From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ffmpeg-devel-bounces@ffmpeg.org>
Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100])
	by master.gitmailbox.com (Postfix) with ESMTPS id 7F0CE4DA0D
	for <ffmpegdev@gitmailbox.com>; Mon, 21 Apr 2025 19:50:37 +0000 (UTC)
Received: from [127.0.1.1] (localhost [127.0.0.1])
	by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 79A9268A112;
	Mon, 21 Apr 2025 22:50:30 +0300 (EEST)
Received: from NAM11-CO1-obe.outbound.protection.outlook.com
 (mail-co1nam11olkn2074.outbound.protection.outlook.com [40.92.18.74])
 by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 29C05687D8B
 for <ffmpeg-devel@ffmpeg.org>; Mon, 21 Apr 2025 22:50:15 +0300 (EEST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none;
 b=vmuiumhue8mQacgaeCoaJ1IfRMiqtHOHiRddAi2T60E5iVGMZVaiIt3QZXiTv9zyO5ue5d2sAJxnu6EzCBd5cA8qLKGJyd2PvbLWsdTNXWljL7mxOPXp4aDRayJkBFHCyV/HSMoYmV91uhPRguOo/58kLg1mWmwQ7UOQHkqOUg1j19F7abAyzqaJxAwRzBeQtlHwheM8Y5Ko7TNR82EOpiBJti/YCdiZwzI0UwG/YgNCETPOGvNu9BUPCri6MojBHd3zcraBj4YGeQkS5tASD23zzlcwRRkRsdAtSYvRoFfnPT4IcehdHHgul3WgHwUPxLI40b/YF37WgqmVhXrpHw==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; 
 s=arcselector10001;
 h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1;
 bh=7LCIpoy4FRMR13mEUogg3+GqXicPp7/zfKMPcCC79xA=;
 b=ByOTfx9Cyk5EJOyj+uJ3RecVT+M0L3UaB70L+eagkgBKh0rTQN7vYOySzRfgeyxvI/mbJCFOBwaBDEE05uJi/KNZX9zFJpJ333wrgjW4dN4M6AHmJkV8Bq6Zsrl3k/6Iq7UC1H/jIkaBXdrw9xLQl6caK5aHXpne9TeEThXeE9JCU6m8wOS/advRv7ELoI+wVbkCTf9yrlzRiBiQZB97wD4r6TB9SThmnYxgu7V6xy8hUfmw/w/YheCsW3NWXA6P48QZLWFEsQ/B626Nbeki9erQ+fU2T1i4dOWym50uLxVAOvIxWbo34/IyCIzokBQFsEJX8UEfpt5Hb/amJQMylQ==
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=hotmail.com;
 s=selector1;
 h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;
 bh=7LCIpoy4FRMR13mEUogg3+GqXicPp7/zfKMPcCC79xA=;
 b=O9IZ3hr7AKQ2Fg2u+qn2hcD0Fe2eu8DerZXSBNzN1DscA9HKZGw9O6bAq9Ol0oQlfZgpey/f4cMPOW2iVssUa2D/Zc1dlLA5uoxENPB2/VY/HdEU8DHyUE+zCFq2C9upX1ku5KNff5UT7Cf4XAavdQbyiu7T9AkQaVR4QIZ8rSXLYDk4LESByiVQkueRJ5vh5QBmgqclpRL6xLwi4jpMFkmPUw6G9vA80XkXLREHlk9Lij1RSoH/uLILLZvH3jMgdu8NRfvLVEPpTwgHjql+HeEa9F41B8ua/FK7kFxy/dWGc/Fotput5fGTwd04kzksWrrjjFlTcS2t7vXlgmFQKA==
Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by
 CH0P223MB0153.NAMP223.PROD.OUTLOOK.COM (2603:10b6:610:f5::16) with Microsoft
 SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id
 15.20.8655.36; Mon, 21 Apr 2025 19:50:11 +0000
Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM
 ([fe80::bf09:8e9:b07f:98a7]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM
 ([fe80::bf09:8e9:b07f:98a7%4]) with mapi id 15.20.8655.033; Mon, 21 Apr 2025
 19:50:11 +0000
From: "softworkz ." <softworkz-at-hotmail.com@ffmpeg.org>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Thread-Topic: [FFmpeg-devel] [PATCH] avformat/dump: Change precision of stream
 start offsets
Thread-Index: AQHbsjBgmnqibkcbuU+WybiBeSgUerOtjYkAgACP6iCAAGYiAIAABERw
Date: Mon, 21 Apr 2025 19:50:11 +0000
Message-ID: <DM8P223MB0365837752CD6F7A92A2F335BAB82@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM>
References: <pull.72.ffstaging.FFmpeg.1745179871512.ffmpegagent@gmail.com>
 <af9371b0-8684-404f-8bde-33e4dcaec555@gyani.pro>
 <DM8P223MB0365F6CE5ADDBFBD31C13CEFBAB82@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM>
 <876061f7-1b81-1d4c-6dbc-51ac2119e2f3@passwd.hu>
In-Reply-To: <876061f7-1b81-1d4c-6dbc-51ac2119e2f3@passwd.hu>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
x-ms-publictraffictype: Email
x-ms-traffictypediagnostic: DM8P223MB0365:EE_|CH0P223MB0153:EE_
x-ms-office365-filtering-correlation-id: c82b1339-1fd7-4e58-aaef-08dd810dba38
x-ms-exchange-slblob-mailprops: Cq7lScuPrnpFX5gNBgVui9nZ6J+Gucr/IFEGlpWWUGsCyMfCKyP9X6OEL6noYoVtjhZHK70/lU4yEKa7ACX+gndxseyQHXUMezmgzf7v5V6zo0Wlo/Qbkoxc0D8yUgi2IEGtveqS8M7LnTGqSPzivn9dLBbkhbtv4UxSV7WUCEPj+v9JkXl59D+rL3J9tbDEsGgYGbZV1X/XMfaDsvboqd4oxafoM/UB8UYciHSnYoOmdn0jyEyTz7whScawRM4ou59eZnhpV+8B34lyW5/NaH+Qpw7XhVaxwf9bgWj04tgfiPzXtynqUNpeYW71Z531VhAaGgX3ct6JUgwDWDQL0fzHR+4wr5GwMAAu/1Xaq8MM1+Q32WahFRNmEMJOvcn35YEuh1NQHP8R9BexxJ6bBYZ/kAkjwiWNYgLs8E631bqkyE3aq2mKbQDnYUNO3e4chonpaf/C+gN9cl/P1fQzmnBTna+Qaaeg0A3cNT1l3zP6+/aKAacVluvHFxqa3b4HEQXAIw+teb2mLPNCpjSz7eOpg06phIOmxDjdyJtXvZQ2A1DJ/bhNJDhm7eJouo3U2I3UTJ3OZNDraZr5lOH9FtRMaxdCy13We1evm5I3B3B1QVLyVhAYBTnPu34VFwv0FUAWvnzMq3HGGere4FcxWlX2VMATltDb6mF5xj4sqgmDd1LINDzJ0UzDI7EwILOmwYFpnOH6PDQSHcLrWXtzFIHhoEn9OqI2wpExVOfpyNGvS6+TSdjRJ1ZdppW+AK4Wy8AMcgYJY6g=
x-microsoft-antispam: BCL:0;
 ARA:14566002|8062599003|461199028|15080799006|7092599003|8060799006|19110799003|41001999003|10035399004|3412199025|440099028|102099032;
x-microsoft-antispam-message-info: =?us-ascii?Q?956blm4AIY1u+mfzHdHDmGQ4tozKrA8/gW732fEuXJZnpXKYr39RRlIykDta?=
 =?us-ascii?Q?gbUXHcLive97nIMIAwEAFhmkVn3iXxCJY4s0VTLG0hdof+C6kSVNvwea7QlN?=
 =?us-ascii?Q?/QmICgSCKP9VEuMDZ82v/vrHL0DEOXxKI5b0cMHtt/hNvEL3hXubDl23gf4M?=
 =?us-ascii?Q?PJ3aMH8uDmePLxfnPqxBmPUsz4xIwuvv/bIEcygBKcivD+zL5YexNDpecmcY?=
 =?us-ascii?Q?m3V5/g2WtNBPt9U5UchqEy0Dp2Q4gBuIltZ7I3x2PpX1v5XcKmhwzJwj2KoB?=
 =?us-ascii?Q?qmYje8uMmr2BjgJ3l09n4HRECLjSHDREZCVdaFSc35+sb4ed9MOJhvSRKWHW?=
 =?us-ascii?Q?wMz80vrXVCNRVzDWblcGIuEaeJW/qgtBEzwvOeoTfcb7f3pLxASOzRSmUI8A?=
 =?us-ascii?Q?Mnsgy87ISimkWLyRPcymYZdepSwed/OSZskp0IsR3egTmJSy0tRVc/Fo09NX?=
 =?us-ascii?Q?4JXeObtMGt5Cx/f0UJjFJ/qB+6fyRFLnwquRnmKMRUNu7Ap4wTxnqlWHYmtZ?=
 =?us-ascii?Q?hjr/gXbSB8DdAyeNVOnJXEveaIP1HbMk5O4/2Uajh4FnRhj4fi8JFWcpg6U0?=
 =?us-ascii?Q?WMNZiBDpOHNL8pCFMskM83c8urSRmFjpUo3sYzQOnwyCwsde7IgxPxk3eJrG?=
 =?us-ascii?Q?eeeYJKB+1dDbDGKv7cS2n3IftArjO1nYZd6QdX4GoAS+THDlloj0dWeNax2O?=
 =?us-ascii?Q?5WBqN9lIelmq/plAuHB/Y3TiLUmuZLx789ScFTf7cHOpxWIJ7WhJphlqfABi?=
 =?us-ascii?Q?6UBcue0d74obm0vCb5K1T3n7Ap1usbjffqwMCaVq9VlDHh1SnmLcyg6YWd8/?=
 =?us-ascii?Q?MFWmusvaD7WSCEBIom5fxe4hLMOSLkBIc7PfDBTKtaJDXuN7p/3GpUJQaT1K?=
 =?us-ascii?Q?OyUKcvdE8QwSEQjc288w0rziP1XZm1us6/PTHav2DmcgFulqiqUS+GDLSJTo?=
 =?us-ascii?Q?V019KnFQn7G574R26z6EswnGEWBRMcTx5fg+8mA4mkdN/v/DQkVbM/X0ajGV?=
 =?us-ascii?Q?ExhnvG0aQ68GJ1VPLbne+bJ2VhucKMFFj6vAmXdPP7edFYT8xyWmZVVcxDok?=
 =?us-ascii?Q?z05U901M8eSkzBZXuklP+orLzJMEIjFh9ulT0GRdd1LVBoMSOFAZlfhx4x8f?=
 =?us-ascii?Q?fYXuAAmPNNqkkt96kMARr6UJ0cJwmcF28o8mOpPyH3UeAUymL6fTWqzfeRAv?=
 =?us-ascii?Q?l8prhc/kTXnDN0jhJk8KhuFDqfFx6FAKi+uysw=3D=3D?=
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?5IS7XiLw4GQf+nG3cc6Cl3DjwuZ+4eK9UK3up1dDHn661S85qbRBPJeoADPT?=
 =?us-ascii?Q?QVgWUpj1iCNq7XXDvz4baki+ecjiImQdTVElpy1ylpqd9VfwnRVipKVDVyiC?=
 =?us-ascii?Q?u3FNFjnBnrEzFKEYi07noIK5qrwpnrqcKQYQv0e0i7l+9J0JCJ/5K3gpzS59?=
 =?us-ascii?Q?iXsxTkiADUs0YoA7PbC2+F+pQicKBjxzU8jZeKjBLHCiSdf1DlamSRAPm2K5?=
 =?us-ascii?Q?NXntXcV3Tjc71HV8J8mVvm5ySwCNzeGP+jwocbMsE1PRF/dWp8r34cZBwhsz?=
 =?us-ascii?Q?P6yh3btl5VbbWJvjOmxJF5Zbnqp2E7JEF9zJ2LOQoex6WUFBh/0Et5Dhjxm6?=
 =?us-ascii?Q?ff1g/sG3KL/qQTOMF7KAhKYDKMurb3rxgLtQLukGbuZG0dNkaEorzCUnQSyf?=
 =?us-ascii?Q?M/LIv49aSc8PxaywJ3EUuZCX7Pl2ePeJUTNkCrbAqLnF8O1zH1DP84jQLBXg?=
 =?us-ascii?Q?PevWNRr4TcvcwmwVDfIvCTzW+gS9aS+yyllR59U3LOHK+VooyhVJ3lrBn6Br?=
 =?us-ascii?Q?P/rkAB3++zXDlTmSMjk81D2blfFVPs+8M/NnxD0CBE4jPVv3qIJhvAKNVrpS?=
 =?us-ascii?Q?r1WraaQDcNwHT7pWyWS3iF6+xrYCyQHuiQaEb5uK6hsXt5qNdT8LUYN2obIZ?=
 =?us-ascii?Q?oyhhLokFCBi71kCQbgjcemLE7IlJqgUviSvowzpmF/3aW1aTdrPwW5Kf4mmi?=
 =?us-ascii?Q?14gtaK0IXPsqXU553943in0t/+LI6/VqpO4uiESJgAlKebbPLOTN8jAmSwLJ?=
 =?us-ascii?Q?tCAV6QBTttCoj6VUq2kgQq+pGfe7Yf9pyDKTbUPZ6HnHQYLVPy8+8b+g7rig?=
 =?us-ascii?Q?wxnyu551yay1aT75spF7613NjTcZpiv1CXxiOWRbYRsIHRVpl/IlmVoFsT+6?=
 =?us-ascii?Q?AVFqx7nTXde6I8JIAv/jQaleZg0QmIoIv9zwnS9F/+wZibpcu7iQ3st42rbd?=
 =?us-ascii?Q?GIfd6gkT9rfXKTlB9CRyrG416SuSRzttXhCLm1PMhwuILxwzMw6wrov7deQS?=
 =?us-ascii?Q?Jt3AoYFXovFAea20ThRkrLFM4W+yx7HzgAQVIPXirEmbjoK8vGijQXPSpySf?=
 =?us-ascii?Q?rOcDZW2XADvbtuvRhdlo8WPW4nhKEfnomp+XN8g+Ja56sxhLJ/4jctiiqI8x?=
 =?us-ascii?Q?FnE87DAQKS3QFDBvOuvIGGlM9N1E/ocjozHbPxo6y79PvDDEfeATG2AG2Gqn?=
 =?us-ascii?Q?F3iWdYp5o0T63Jb7uUMcRv+7Ukud+cPbZZy6Ax/AvqXaFdOdrTWCrUeZcsA?=
 =?us-ascii?Q?=3D?=
MIME-Version: 1.0
X-OriginatorOrg: sct-15-20-7719-20-msonline-outlook-92255.templateTenant
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-CrossTenant-Network-Message-Id: c82b1339-1fd7-4e58-aaef-08dd810dba38
X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Apr 2025 19:50:11.4969 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa
X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0P223MB0153
Subject: Re: [FFmpeg-devel] [PATCH] avformat/dump: Change precision of
 stream start offsets
X-BeenThere: ffmpeg-devel@ffmpeg.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: FFmpeg development discussions and patches <ffmpeg-devel.ffmpeg.org>
List-Unsubscribe: <https://ffmpeg.org/mailman/options/ffmpeg-devel>,
 <mailto:ffmpeg-devel-request@ffmpeg.org?subject=unsubscribe>
List-Archive: <https://ffmpeg.org/pipermail/ffmpeg-devel>
List-Post: <mailto:ffmpeg-devel@ffmpeg.org>
List-Help: <mailto:ffmpeg-devel-request@ffmpeg.org?subject=help>
List-Subscribe: <https://ffmpeg.org/mailman/listinfo/ffmpeg-devel>,
 <mailto:ffmpeg-devel-request@ffmpeg.org?subject=subscribe>
Reply-To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: ffmpeg-devel-bounces@ffmpeg.org
Sender: "ffmpeg-devel" <ffmpeg-devel-bounces@ffmpeg.org>
Archived-At: <https://master.gitmailbox.com/ffmpegdev/DM8P223MB0365837752CD6F7A92A2F335BAB82@DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM/>
List-Archive: <https://master.gitmailbox.com/ffmpegdev/>
List-Post: <mailto:ffmpegdev@gitmailbox.com>



> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of
> Marton Balint
> Sent: Montag, 21. April 2025 21:32
> To: FFmpeg development discussions and patches <ffmpeg-
> devel@ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH] avformat/dump: Change precision of
> stream start offsets
> 
> 
> 
> On Mon, 21 Apr 2025, softworkz . wrote:
> 
> >
> >
> >> -----Original Message-----
> >> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of
> Gyan
> >> Doshi
> >> Sent: Montag, 21. April 2025 06:51
> >> To: ffmpeg-devel@ffmpeg.org
> >> Subject: Re: [FFmpeg-devel] [PATCH] avformat/dump: Change precision
> of
> >> stream start offsets
> >>
> >>
> >>
> >> On 2025-04-21 01:41 am, softworkz wrote:
> >>> From: softworkz <softworkz@hotmail.com>
> >>>
> >>> Changing this to 6 digits to align with other
> >>> printed times
> >>>
> >>> Signed-off-by: softworkz <softworkz@hotmail.com>
> >>> ---
> >>>      avformat/dump: Change precision of stream start offsets
> >>>
> >>>      Changing this to 6 digits to align with other printed times
> >>>
> >>>      Signed-off-by: softworkz softworkz@hotmail.com
> >>>
> >>> Published-As: https://github.com/ffstaging/FFmpeg/releases/tag/pr-
> >> ffstaging-72%2Fsoftworkz%2Fsubmit_start_offsets-v1
> >>> Fetch-It-Via: git fetch https://github.com/ffstaging/FFmpeg pr-
> >> ffstaging-72/softworkz/submit_start_offsets-v1
> >>> Pull-Request: https://github.com/ffstaging/FFmpeg/pull/72
> >>>
> >>>   libavformat/dump.c | 2 +-
> >>>   1 file changed, 1 insertion(+), 1 deletion(-)
> >>>
> >>> diff --git a/libavformat/dump.c b/libavformat/dump.c
> >>> index 8c7db7b275..1bd0424f3d 100644
> >>> --- a/libavformat/dump.c
> >>> +++ b/libavformat/dump.c
> >>> @@ -680,7 +680,7 @@ FF_ENABLE_DEPRECATION_WARNINGS
> >>>
> >>>       if (st->start_time != AV_NOPTS_VALUE && st->start_time != 0
> &&
> >> st->time_base.den && st->time_base.num) {
> >>>           const double stream_start = av_q2d(st->time_base) * st-
> >>> start_time;
> >>> -        av_log(NULL, AV_LOG_INFO, ", Start-Time %.3fs",
> >> stream_start);
> >>> +        av_log(NULL, AV_LOG_INFO, ", Start-Time %.6fs",
> >> stream_start);
> >>
> >> The camel case is incongruous with the formatting of the text next
> To
> >> It.
> >
> > Hi Gyan,
> >
> > as far as I'm seeing it, captions/labels are title case and values
> are
> > lower case. I would consider "Start-Time" to be a caption/label.
> Let's
> > look at an example:
> >
> >  Stream #0:0[0x8fd]: Video: h264 (High) ([27][0][0][0] / 0x001B),
> yuv420p(tv, bt709, top first), 1920x1080 [SAR 1:1 DAR 16:9], Level 40,
> 25 fps, 50 tbr, 90k tbn, Start-Time 32476.588s
> >  Stream #0:1[0x907](dut): Audio: mp2 ([3][0][0][0] / 0x0003), 48000
> Hz, stereo, fltp, 256 kb/s, Start-Time 32476.706s
> >  Stream #0:2[0x908](dut): Audio: ac3 (AC-3 / 0x332D4341), 48000 Hz,
> 5.1(side), fltp, 448 kb/s, Start-Time 32476.643s
> >  Stream #0:3[0x909](GOS): Audio: mp2 ([3][0][0][0] / 0x0003), 48000
> Hz, stereo, fltp, 256 kb/s, Start-Time 32476.674s (visual impaired)
> (descriptions)
> >  Stream #0:4[0x962](dut): Subtitle: dvb_subtitle ([6][0][0][0] /
> 0x0006), Start-Time 32476.588s
> >  Stream #0:5[0x963](dut): Subtitle: dvb_subtitle ([6][0][0][0] /
> 0x0006), Start-Time 32476.719s (hearing impaired)
> 
> I think Gyan means fps, tbr or tbn. These are all lowercase and
> abbreviations.

Yes, I understood that. But these are units, not labels.


> The one-line stream info should be as compact as
> possible.

Agreed.

> So we should make it shorter, and we should lose the "s" unit as
> well. 

I'm afraid, but omitting the unit makes no sense to me.

Why?



> Something like:
> 
> "in 1234.567"

What means "in"?

Maybe "offset" if it's gotta be short..?

Thanks,
sw
_______________________________________________
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".