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 74E8E4E273 for ; Sat, 7 Jun 2025 23:37:46 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id F159668BF94; Sun, 8 Jun 2025 02:37:41 +0300 (EEST) Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05olkn2042.outbound.protection.outlook.com [40.92.90.42]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id 3FFDD68BA13 for ; Sun, 8 Jun 2025 02:37:35 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=X5urEGYi0PR7uaYj88N8NImlkGWHmf6NCYCBNpBlB1RcfCRR1mCwB00ufz5mBKmtH0IBRjdXX8SGPdLhhYvxnLJ6u4t3r2GK+sifCyV2GXyGiIway5OHWIitue7sn79VezThiD/hab4Cg+opLPGLp2s4GiUpTVDCacD79z1fc+PW1nWprdP6LIpaU2+hKu5kvos3P/zRnprFxjpG3QlehCaZU/Wvlwhjy3UeBYYzwqDbeUxrz/gSFZmcjX0d+vVcR7U4ufnROvxwKErxOKokB/E8dOSI8QjIp2bnsjKUzI3hINwXjE6yRIMEj7unyw0IVzoMuokuVgUMk2m4E8qjyw== 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=DLTpEtbtFTrRVroLV+zN2HqILL8HirPBz+bCS7mRWdo=; b=Da+d/DHAsXNOCzlzIdOxYghjyWBzdqv9XfG5/Y/C85FH3OOxy9kyCIkN+etgnbBw2Iv+E6Si7UNN3VExH68wLpUF8k+eIUDqPjsOFt1w37gdGWNYbI75/aqK01dqAFkcu/pCtS/MbLI+s+3DetlIDkPmzW+W5CrhxXAxVCx3pB9bUBwfcqmf77ny2T8+Zm+9gooRBduJBgXuEwmDhzkiyPZhlJj24yPRWTLstnkP/ocC1nIA+j3I4PSDy2Td4Ig68azLT1vO7HirCxHxpDLb7IZPKDR2rJbazzXzHR2w3rxW1sbeFfEFzNtIvzi8JNazodL+f+ipKZXBZBUYDiXyeQ== 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=DLTpEtbtFTrRVroLV+zN2HqILL8HirPBz+bCS7mRWdo=; b=CIHmZOKGGF1s465ZkSXgT7PfkAJgmr+EkFLdzKd6caewbblhWqpNV1HHrGpc2jwyvMbRpEbFieF+pNkSz1ie2juvHYjCtPJWi5KL8biDPnYHA9nRnkLXQU3PRo9yR688jF3UymaI44MzxNInPXg3SJkdKX5r1y4V97Meda7dTfd+r6WIYcMVDDpzfdykcafqXnjKefY04gcj2D1kEHUlx/WZYVSwB1s/GVBGDf8if3FJ4twgPBYC70nvzUKkH7EMkxtBhE0hbFDqu2/SqVEn68uQHUWGBn492At1FW52cP6HlNbKtlPxlonnPYe7PaR1gRSVRfoCSHMK9s3RSJxjwQ== Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) by AM8P250MB0011.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:3da::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8792.36; Sat, 7 Jun 2025 23:37:34 +0000 Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::d6a1:e3af:a5f1:b614]) by GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::d6a1:e3af:a5f1:b614%5]) with mapi id 15.20.8792.034; Sat, 7 Jun 2025 23:37:33 +0000 Message-ID: Date: Sun, 8 Jun 2025 01:37:32 +0200 User-Agent: Mozilla Thunderbird To: ffmpeg-devel@ffmpeg.org References: <20250531133215.64025-1-michael@niedermayer.cc> <20250604010512.GN29660@pb2> <20250607232905.GK29660@pb2> Content-Language: en-US From: Andreas Rheinhardt In-Reply-To: <20250607232905.GK29660@pb2> X-ClientProxiedBy: FR4P281CA0388.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:f7::17) To GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) X-Microsoft-Original-Message-ID: MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: GV1P250MB0737:EE_|AM8P250MB0011:EE_ X-MS-Office365-Filtering-Correlation-Id: df835b05-47ef-46ed-0ca7-08dda61c46f7 X-MS-Exchange-SLBlob-MailProps: AZnQBsB9XmpKEBsvou3idazKhbklnsCkcGRCffIvImpOSnrlk3Rt8nlR8lRvCD3sW2NEPl9ttptvt7mK0HXIzTG+qH/PK9YKFsS8TliU3us6Al21LurQMi20/G7SwzTiZLiJWMfooJMh3cBaCk0vuP6rJ/z8bRVgRaCJFHPOPLzfSeDRv2zj1pnDzqOSG5UNeeo5hSSoZqkL0TxcNTfvMkK603+DoRuG4ZbVtpXN3nxd96X8Iu7oKfZlbP99rvxdpCNSjDK4ci+oQ1N9YDt4OiEOCaehCIyQrp9dx+vEWDLcyU63+pKfnLIHp9a51bTownbv3w8F2b19UwfWBuC/q2GrKfovSFl/927i9kkXTfmNdzbwuyfssELkLL7YJLmr3ZO/A0UBQX4G8bjJs32zV3a95/gIRser99ohHOBvIknIHo1/o/1hBufY32BYPPN5aYLW6j0BI83zV798oYmls59knJYmr2CFm2fG2QrtNlVTtALPIvprebYvACCoxo1gKbLYW+hU24ygjr7PdW4J43kcgzmKpM6uqkNJMaApyjtFmBcgqUJrUOUKqnwvHb5qqdZ2b1xqnKN86EIkRcgahIbeDmcxhMqR1gIYf31Hp74uMVPLMWK+yAcTOOC6Ot4UacHrR0hf+dTaLwNj9n3svLnk8qVNuhDuN8+GA0MX/kOZSmDvMbMVXmi38KyCC+WGqoYAmkxzmAEos3NQ29Yjv21UGEqc+n1rar/0gPvsG5aRoVaEIMCyxGROSzeh37BBIGPOk5QZPvg= X-Microsoft-Antispam: BCL:0; ARA:14566002|6090799003|8060799009|7092599006|5072599009|461199028|19110799006|15080799009|440099028|3412199025; X-Microsoft-Antispam-Message-Info: =?utf-8?B?d2N4VWRmcCtFem9oQWVCS3VuRFBDS1pjKy9qbUYxUnJyOEF3UG1kWnpDNU83?= =?utf-8?B?aHRmRk5nVU1Ha2llN3h0dDFVaWFXQTZxM3VobXRlbUIzb0Y5UExEWExIS1kz?= =?utf-8?B?Sm9tLzloTjBSdHNtb3RreTkrSWptYjNlbEswSkZsRmp3b2lSd0p1ZzBrY0t2?= =?utf-8?B?RmpzQ0JJaGJ0bHBsMTQ4bnhCYStZdjEwemxiVFB0OFJlWWo4YzRRU1pwUWdW?= =?utf-8?B?bkVLcSs0UER4ZWtZUXdIcHJlTGx0ZGc3NExoSlZ1QjcrZERybmd2c3orVmhW?= =?utf-8?B?SUVqNGFFQW5TbDNDM3FjbDIwcXZ2NDJoSUFGTUl3SEQ2TXdPYUN5bWhCTzZP?= =?utf-8?B?cG5FV0lWbHBUUmxKa0lsMCtRYVEvMVpLdmpUaTl6TXVhQVcxNHpsQk8vM2VN?= =?utf-8?B?Z2VJRUdTd3FaU2p0NVRld3BZMUFNRm04ZU9ySUlZKyswRGMzM1FXaGNLdXZU?= =?utf-8?B?MFlncE9RZlp1V3NDWEQxZ0Q1bGMzNkd0WDByMTRCQ0pPa2NkZGpURFM1QWVt?= =?utf-8?B?YkJDanRCSjE0Z1dycERGeEVScmdtNW5STTBNZE90aHg2enBuM0J6UE9xcDU5?= =?utf-8?B?WFllMW50b3pnbVZ0cEtnOTN1bFM3bCtJSjQxOWQ1anVZS25MTHhHdEd6VFRi?= =?utf-8?B?eHlkQis1N1oyaWNvbGltMlJTdi84VGFhVkhrbjJxSElQVnhUNG1hbWxudGtE?= =?utf-8?B?dUVtaXVIR2hRdDFRdENGQWRGSXRjbnBLaHBlRHdhR1JiQmZYSFVlNy9LZWlF?= =?utf-8?B?c2s3aWNybjhCQkppSEw1dUhlNllEZ0dYOEc3bUhDK1dwK3J0RTgrbUE3d0tZ?= =?utf-8?B?Sk1mY0grS1lWMWVFYmxIbTFCSVk4K1FWOFRPNlZoYjhvMXBmUWEyQ3RWUlhN?= =?utf-8?B?OHpDZFo5RnpBVFVRQmtZbnJOWlRXcnZuaklIQzYxaTZ6NGYvQWRIZ0EraGRQ?= =?utf-8?B?cVpsVERPMHlSN3o5dHUwTGpyTm1GclVGTGx5UFkzMWN6KzFoU013WURQbGg2?= =?utf-8?B?VGV1cVdPU0duMHptempoL3J5V0ZjdEd2T2hWcHUzMjRFQUQzMDhKUmxrRVpi?= =?utf-8?B?YUtjNzBYODlsMUpRdE4rQ3IwV3Eyekc3MDhTTG5HMkVmT3RvQ0VjSGRYUGRX?= =?utf-8?B?aTJlNW0zSkxuaUw4VG9QTkVyVjVpVmZVY1NDSG9GSkdpNTMxQUFFQVgvRkl6?= =?utf-8?B?OWN1VXhLam45ZEhrSFBYWnp2RjFyR0RBdEZ0VUdqN1A0b1JCbnNYcTRyYzll?= =?utf-8?B?YklvMVdoUDgvVkU0RzhYa0lReVVDUVp2elVIaXpnMDBXQTBiNW5veXhacTg4?= =?utf-8?B?MzJtajRPSkovazl3SzNReFdPVFFaTXdmaG92SHorbHBzekxWT2VPVVNLVFVX?= =?utf-8?B?L1FHTUxQeG5WSnFlSzBBSU0wYTFQSDFQWDFMZUxxRzA1dTlSK3JmTEU3OTRn?= =?utf-8?B?T1o0Unh0bGhuT0NIaVA4TnFDOWFKejNNRkNmZHh0WHBRT3FCZDQvRHh5WWtr?= =?utf-8?B?USs2cFNaMUlnMmIxWG92Z2pVZmM0b0FIVFFYb1ZJaWpVanBOK3VHUU1hcy9p?= =?utf-8?B?dDFldz09?= X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?aFExbUMyZHRhNFNYeHp3aWwxbVlXbUJWK0NjYy9Mc0pCWUpHRmpjcFFCc0hl?= =?utf-8?B?Mm1BREdWbWZqTWNVUVdIV2NHdnlZVnFUK01TbUFDc3VCaitMNldTdEZ2SHhm?= =?utf-8?B?Vkt0d2c0VlhDK0tMUlhMRWZnalJGOVVmbTdyTHNTcTh2a1BGdUl3OVNHNmFx?= =?utf-8?B?SXRtU2ZuR2lUTjQ0cGFHcWp0RFdPWHFWNXA5Y3lwYndSbHd0TCtQWm9EbU1P?= =?utf-8?B?cHNidFJ5WkUzcnkyUlc1Z09OcXhLUnFIREFwTndqZWZTeW8xbXJxWGhJRlNO?= =?utf-8?B?akJSamFYS3BUMmRxL3Z4Z2lIMUY4QjU0V2d6aEMxbWhBYkhMK2pKcDNVbGpr?= =?utf-8?B?ZGZ2OHBYNlJhZjliZWIxQWczNWlhWi82QnluUnJpcEZoVzZmWEk3YzJIWDF6?= =?utf-8?B?TERraXVkWGZLVlRNeXBxV2NsRTE0RzZkRXRaN3dONE9vNFRPeHhiZXZPYlBQ?= =?utf-8?B?cVZQRTVYZ2dJUFMybnNOcm9WSEo5cjU4cXZhMmFwODgyNXRqeit3RFFUZWJs?= =?utf-8?B?RWxZS04vWk55STVWUTd3TTBVcVpNQTNIMjRPTEpVcGxwQmNmTEx5dUNzbjFQ?= =?utf-8?B?cm9mMjRvU3JRY1ZmNzdweXlBUDFESDdvS0h6Rk1WSmNRdXJKOU85Sjlmckdh?= =?utf-8?B?aWM1U2dwS2VmUlMwc0ZsODIxUkNub0ZRdGFPN0tyY3ZOY2JJa3d1NXFVMkdY?= =?utf-8?B?RlBuZWpRY3U3MjFIRmQ3WnZVaForN0VqN3U1ZkFKWnpQY25PMm5aaDRSTGwx?= =?utf-8?B?RitRQkVCQnpJaG5rRW92cHhLVkkwTVJyWEkxVnBucVB5aUM0RWlFUzFjSEdt?= =?utf-8?B?YW1Wbks1YzRnMlFUSzRGbCtjc3hpcXhiaWZPcitDLzZBM3dSUGxucndvUFZB?= =?utf-8?B?MXNWWTRrbVNka05MZEUxaW5iVi9sUHhKWDR3bXFPOUdJcDZOT3FEbDlCc2lQ?= =?utf-8?B?NW1ab3daK1Vqem55dFk4ODhIczhmaWVHYm90VFZkZVZndlBNZ3Vla0Y1bmIx?= =?utf-8?B?NXBuSm1PL3AwTU1EdkNXa2U0OCtuSzN3Wmh1aXNjcnlZTU15am44RHJNRldq?= =?utf-8?B?d004QnpVY2Z3a1NQOU5hektLL1pTbEpjOFF1R2VIY2tNTnV1NSs4T1NWeVdz?= =?utf-8?B?RjZmRUFOZSs1NVExRVFBTGowY1htWmlWWjRJYzhqdVJHYUkwNXBtRlFqMjB2?= =?utf-8?B?RnVuZkVXNkZlR0czR05qZUhWbVMwRVlzOTN0VGNMM2JHZEF5ZGdyd211TXhj?= =?utf-8?B?amJhTllRN2FkTU9MUGZjUWpmemJwalNJemtKRTFrcEZQMm5jTG9uUjJXeU45?= =?utf-8?B?dnBjZUlkUThjQjlvb1kyYkNmWVEyTzRMMEFaVjJ1N3JpY1FTZ0VWVGt0dXRw?= =?utf-8?B?b3hIQ3RNZWFhYUQ2Y29jaWhnTngyV0E5ejIweTFOQ2FHU250SUdBZzI5dWxM?= =?utf-8?B?NWtKRDJCenB0MzdjaGZaSXVKaWE5U3lOakhJQlV1akQyb0oxQm5oUnI3UE11?= =?utf-8?B?NW1NQWY5b2JuQkc0Z0sycHUxczBOWTlsWldPb1MwRXVTNU1DVDlmNVAxdFlu?= =?utf-8?B?a296dXNrSmJrYWNWYTJzWUdvRm5ESTlZM0MyNWVHVUNVdUlybWNCMUI3bURD?= =?utf-8?B?cUVHWFU0UVRzL3k4b3R5Y0dCM1ZYRlhzVFNITW5ZMUV0cmU3Wld6Ukt3dXJl?= =?utf-8?B?RC9lTU5pL1dhMldPWXVQVUJ6YXRDbmNycGtRL29yVEkzT0Y1dTdEM0hzRjdB?= =?utf-8?B?bU5ubVdiSHBzejVQKzNTaEQ2TzBhL0FuZ1N4OUt4d1psbWZuSGpaM0tuY2E1?= =?utf-8?B?OHRsSDl5K3krdUE4VUJxUT09?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: df835b05-47ef-46ed-0ca7-08dda61c46f7 X-MS-Exchange-CrossTenant-AuthSource: GV1P250MB0737.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Jun 2025 23:37:33.8558 (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: AM8P250MB0011 Subject: Re: [FFmpeg-devel] [PATCH 1/2] Replace FFMIN/FFMAX by type specific macros 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: Michael Niedermayer: > Hi > > On Wed, Jun 04, 2025 at 03:05:12AM +0200, Michael Niedermayer wrote: > [...] >>>> #define FFSWAP(type,a,b) do{type SWAP_tmp= b; b= a; a= SWAP_tmp;}while(0) >>> >>> 1. I don't like that you change the common case (namely the integer one) >>> and thereby making the name longer (and making this patch huge). >> >> I can just rename FFMINI/MAXI to FFMIN/MAX if preferred > > are there any objections to FFMIND / FFMINF / FFMAXD / FFMAXF ? > if not i intend to apply these in the next days (unless i forget) > I don't want them to be public. >> >> >>> 2. This is supposed to be used to provide optimized platform-specific >>> floating-point min and max functions. How is this supposed to be done >>> with your approach? Would macros.h get internal plattform-specific >>> headers like intreadwrite.h that are only included when building FFmpeg >>> to override the ordinary macros? >> >> I just intended to do the work that noone else wanted to do, >> which was replacing the macros with type specific ones. >> >> These can then be moved to whatever is the best place when optimizations >> are introduced or replaced by a static inline function where thats possible > > thx > > [...] > _______________________________________________ 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".