From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 804E44CD08 for ; Fri, 13 Jun 2025 01:59:28 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id EA31F68C434; Fri, 13 Jun 2025 04:59:23 +0300 (EEST) Received: from NAM12-DM6-obe.outbound.protection.outlook.com (mail-dm6nam12olkn2106.outbound.protection.outlook.com [40.92.22.106]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id 586B26803C0 for ; Fri, 13 Jun 2025 04:59:16 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=yKHhJdB4eIGH06WW9jW/f3uN4k1R2EdxxHLVhgCGcel4BjUmrhoEiNfYs5C0zcyXX1CfTca5et+JwVpZDhk3QmNmx+aW+iXaH+zcipYjjeMFb95qic0pvYKqKhXyPtxOHlK/5zj9boue7ZGje+419ovp0XsFR7dDbEOalkWqOR12Q5xvzIqNgTC+RNla1p6PCj5E6wGYaV7/UOsruK5N7cSlY+9EETTWGO24GEz68u1YV0R1dLeYVrGcsDKigmp46kJSZK3ljoi8o357O/jnxpuWNEaN80yNaRvrE0jvgWeX6xeb3BlSMq7/HBERdib94s6D9hodzA3lM+93G9/QTQ== 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=BpkYlgYgTq8T1ZMoygKYKV8YQJzswbGVkTPDS7iFa+Q=; b=aFmifH1vmlYKkzjqdIq+mS4RhgiqZjggmxFFtmzaSpVGsdSoyet/ERWEY6FmkYLBQO3BooMDR7fbexa4gqf0lv4qRfmgfvTKbaINvvz5WPXfjlSgJYJNqwKAkqOKlttbyPvL85FGV1IJyTouVyvLA6Os8YQqBs/idTpeBaGptbU/BgrC83KBJsWaD8WB3DnB40r1ZlHxxW6VHfmsNReK8So2PHTHX0NUSBv2cgKO+4knhjyBjt8ivc+y7X+PipAqJR+fkSb8z9buH3ZmK95bFh5aa9U0eFScDQeg6s1zvMFvCbdnOppNeOvu6cBHkhJhEzyFiLnTeMdLQxK0anePcg== 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=BpkYlgYgTq8T1ZMoygKYKV8YQJzswbGVkTPDS7iFa+Q=; b=BnZY/ICPJBoholQfbnoyuwyRTJgD5trYJIGRsAzjDTS2jFa0QZwqVKWlRQc3+cNOXRfZDjr//nPdHIZh1iINUtdVB0HLoaQ2n7BkPZQhsw4zsTdJT+N91kz0UXawt4F3wKHdiSLNjrnTj0LljI70ZLVY4+CLunbkCRlyVLUSdzwmEYNm0bdDbIESPqrvHl9yUJUVUeqmemTjtTtUeI5telHanKpaTUwyYAM6J5JBbm1xVdjd4dsS2c/j2Fu5NzuY0Gssb3mOFGcsiUrDVPeQk0A4wCUiTFUQ26q5DtxVfoMJACL1hZ0iYABK1I7AtP8u3wDMV04EaO8qMBrkOwkCLg== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by MN0P223MB1229.NAMP223.PROD.OUTLOOK.COM (2603:10b6:208:4c3::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8813.26; Fri, 13 Jun 2025 01:59:12 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::bf09:8e9:b07f:98a7]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::bf09:8e9:b07f:98a7%6]) with mapi id 15.20.8835.023; Fri, 13 Jun 2025 01:59:12 +0000 From: "softworkz ." To: FFmpeg development discussions and patches Thread-Topic: [RFC] Code Style Questions Thread-Index: AdvcBWS9Dz00tXNHQ/mmaSrAQgQwPA== Date: Fri, 13 Jun 2025 01:59:12 +0000 Message-ID: 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_|MN0P223MB1229:EE_ x-ms-office365-filtering-correlation-id: 4c6439ee-3dcd-4927-4f33-08ddaa1de4cc x-microsoft-antispam: BCL:0; ARA:14566002|7092599006|461199028|19110799006|15080799009|8062599006|8060799009|3412199025|440099028|102099032|40105399003; x-microsoft-antispam-message-info: =?us-ascii?Q?q+FLbQYxX3cr6CwrCGvkD6lq2QqgxrtGJ8vodzuhJD01iEODcLa6AMkv5aQI?= =?us-ascii?Q?3jEuPfauDg2VwH9xHV47SLCYGqRQQE4swLqsdmz3PeTW3/8emAcgt0MXUGoW?= =?us-ascii?Q?Tq84NM+nbCp51QmJoWe7BhBKXekmDje5fw61Y9F3Tz0i2liMdDTs51hVd8wX?= =?us-ascii?Q?uisYgGWlyD+6iuFyiDnmq0wGS8C7Ea6Qr+YWH4/oFpQ0HoiDd7ssUqAxJwaR?= =?us-ascii?Q?zEyPyQZUaRuW78eUIX4qZl25oEx0HYx/OBdyqicMzajNSlXRFHVanXVVfDH3?= =?us-ascii?Q?RYPuGvOZ0lesyQ3MJZY71DIRRPn29Y0H43j5KVvfNWakjpegV3Mnda536fC8?= =?us-ascii?Q?4S4fj8QGVZ4XchzPM9wffG6OuqMfQTS3d4LGsdxZ9XPgGO5YBUEhfCGfYh95?= =?us-ascii?Q?sL/VITuGN3SNL5b/lbmnF7TQhcSZ2P6BnQcOXVZB3yfF7t5hbAFlnbKQMCH0?= =?us-ascii?Q?U//hKdVmtz6u7MEImODzuhczpdMhMdCzoz6tqxJoD8acfkpLss1iiP2qCxgK?= =?us-ascii?Q?J3Bl4QO9AVmkpinBIJbeZZAlSorfZDdhd9oeRc0fHe9m+LYbUrlCTKbEwPmh?= =?us-ascii?Q?EaFe4dG4t5K8IRz/SmFrlp1iMRjhQCnWhYm/ymF2hqH7te0ONyrLnw1mgq/F?= =?us-ascii?Q?H7ot5xSjRUCJhfDCdFkODbO3l+8NwpGDsNJao3fi8W/9uhtfRiYrgvVO3aWu?= =?us-ascii?Q?ppqKjmfCi9c05VFObujsjmIwe+xj4MflHAIBTugk7Us/qmJHiesdzHbwK4Du?= =?us-ascii?Q?RKNDg0wpsu6Y5OMjbeRYHmI5aCs2puAHteRZaIwd+FRxpZ8jzcyFJPiSRQBc?= =?us-ascii?Q?deB9e8h4zPSxO9CIR6b4+lixrYF7bQl8tYMvxH5Pe6T4LwnqdwLFTKtuHrfJ?= =?us-ascii?Q?Lc8n9auVTdlHouGHbJS6o6uFFUV6E/+xvL7bVL7xw1LEln9cg8RrHlciCWNz?= =?us-ascii?Q?s8V0ZLsgn3Op2m/AUOluqY+ad/A2LJ67palJWzmKmoKcUk2XjNz2SlIQRKtk?= =?us-ascii?Q?/iT+9Qogcny6sOFT8D6zfBxDO//HgwDGbRArpgZeTvVGMXCYXx1cS0QGRAxe?= =?us-ascii?Q?H0MkLU+d5DCfltBJuAfMRO0VI1lMybFma6VU2Gh5kqb/1FUmLgUKGjUbtGyW?= =?us-ascii?Q?0ivJuBVdQQdnEKJPi6uhi2PaE4GUYtlTbg=3D=3D?= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?GLTL8AhMVlkN9+G3DQZbv34m6YMCvSPF7uLPz6Z7GaES5hCrvODDo8Vm96JO?= =?us-ascii?Q?nZpkS0Yg008KCNocBNvBu3tKVW+UfbtTMqxtdpHSZtZlt15Fp+9W39TEjUsw?= =?us-ascii?Q?YQIszYfkekKBKEVhWtadGR/X2nmSG3M3BKJ49dmvxkBgwGkJyHjpOc3WW3bF?= =?us-ascii?Q?tgw55KAQ+v74zxcTT9qn4O1hE8Ctxg/oOAjREG2HTSJ1SgZf2zTSPZPwLUmQ?= =?us-ascii?Q?WV5LDxZBgWh/IltAQtx6qdywBZ8/TOBY/mnMtEG9moeiEKTETW0+l7Ow2Egl?= =?us-ascii?Q?C5F/tf9psD16TqUSzQT2j8zvM0xLJxTbbziCpJGYYpOZSDYxMoSCrB5HgQ8w?= =?us-ascii?Q?5CxgwetW1zmn4gtPpUx3ZRBCeqG9YyKQAUiC2xn1icnztxzb48rvmDJgrVDz?= =?us-ascii?Q?uFdGpyGdaCzyAU2Cq36MfMsBVioFnvWCBfRdkSgy+m/9P1da8RcsJ9sXmttx?= =?us-ascii?Q?7SBpUQWC6DL9kUrb4iZfILL5JyPO7XrxSjrh9PcajpAr7AOT2wfiTI/AX1UM?= =?us-ascii?Q?yeEzp7sn++qJ+9jhtJUABWHHEZsrkNQrWPuqmYN8buf/N2QB6VEIK71uSLbV?= =?us-ascii?Q?SuwbgrnuleZfrEV1XdwvqGs00yOgo6S3sToK34QhTmI2mYQRz8OHV6XmgsXo?= =?us-ascii?Q?Zej1cijb8T5Qll8DRIRmu0dY59AutIXz1jptd1I5xe/76zKkS/b4npnrJLB0?= =?us-ascii?Q?MtNkvgA/cw3LracZYStX9addbfqnI2umAalHn86At0C1iNEn0p1b29afYM6g?= =?us-ascii?Q?AXrUmgDFchjIeodQVmBZduFZeaJgYjHLHxEEEnCDS5iGRXuuV9mjCOtpV2Sf?= =?us-ascii?Q?Ty6ZiGqJApmH61AO5PVqyrc3bVyB4fVVyRa88y0KmE5g1CJjY+Hzduc4k4AU?= =?us-ascii?Q?H/oWpSUqBWej+GxVVB0shkDYp0+RsFxnSenCZi3SA8t7A+Uh0jkK7TFw3p+Y?= =?us-ascii?Q?oELvGJCcFq+qIC9N5QFeSLKRVtGe3bG4sWUF8nYdPd9Yg+uKMVRvSuyvylem?= =?us-ascii?Q?6IcDLARLhInULRt02MhBLYr9fva4a3XE4Tc4KAqmSUMmN2svDLDSYY18ML/7?= =?us-ascii?Q?lUcO1r68FFVoISPBJu3ijzHGP9dq3iVRmnPmb/6UTP6F7LLEAN0ARQnpd2lR?= =?us-ascii?Q?I/Y6DTuf2tM1/kDFF7hN6ljTyko2EFs1jD8XqZcdg7Z9vEAsWTcFCaZLrC6x?= =?us-ascii?Q?HSuL2T/UbDtgMDDZLgsnEgJSx2ODSN/HnbMBH0Gihb5UOyxnt7NA3ylA7wY?= =?us-ascii?Q?=3D?= MIME-Version: 1.0 X-OriginatorOrg: sct-15-20-8534-20-msonline-outlook-c7cf3.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: 4c6439ee-3dcd-4927-4f33-08ddaa1de4cc X-MS-Exchange-CrossTenant-originalarrivaltime: 13 Jun 2025 01:59:12.5829 (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: MN0P223MB1229 Subject: [FFmpeg-devel] [RFC] Code Style Questions 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: Hi, I have some questions regarding code style. 1. White space ============== Do we prefer this (float)(*times)[i] / 1000000, (float)(*times)[I - 1] / 1000000); or this (float)(*times)[i]/1000000, (float)(*times)[i-1]/1000000); 2. Multiline Ternary Expressions ================================ Do we prefer this: end_pts = (INT64_MAX - end_pts >= seg->reference_stream_first_pts) ? seg->reference_stream_first_pts : INT64_MAX - end_pts; or this end_pts = (INT64_MAX - end_pts >= seg->reference_stream_first_pts) ? seg->reference_stream_first_pts : INT64_MAX - end_pts; 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".