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 4D8C7430FE for ; Fri, 20 May 2022 18:03:46 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id DCBAF68B47B; Fri, 20 May 2022 21:03:43 +0300 (EEST) Received: from NAM10-MW2-obe.outbound.protection.outlook.com (mail-mw2nam10olkn2044.outbound.protection.outlook.com [40.92.42.44]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 87DD268B43B for ; Fri, 20 May 2022 21:03:36 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=BdTTDrZDNZpmLHfU6qKLsTRetS3Tahj6iXA5PY4lvzY1TTgdhiY+aDaqdkQaqVdugNyIKC2Q50rodOpammadQ6u9HXcnD22kbHtvkQ+7pR40Cu4Uyx0/GKwfErHso2bNxs/tSb4FeHze+4In9I7yCmBiI+QSy4WyCpQaxNytPmLBmQxKWf3Una4yu2N0DM/9Tm3KPuuWghuw1LHOVD/lc2XLx87clfT/YHg4bFbzvIqlblsFhw3n5k3EGrNHqxo2pzmTFXfR8x1+EKnEeAcveNMXKd65caiiXgMXbsF8aRpiBl9Z/AVZHcvMX4bCEzH+CUF4oan7iLOtvENfFg1wgA== 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=Qc37kZBNgc1Gr8OKw0BTKcwnNqHGtKJE6txXLZcjPR4=; b=S/40QBi02UWeTud3iFQsv0i5TlcqETolFhp6knDhJevTwfFqxmY45AClY5ffqQd9OlQ6f8mXJwAIzUh0t7jhURJMpKIKb/7DJ7takyP6IlnEESlSYUzuDw3Zxzs1OyUXeB2/Gyd0HVhlAP8vjjS0ZQ8+xlF1SubTU1UlXM1s2ReFuQ542fpO2aMtLwI5nO5ne3xnmr8+8BIeZB5Q3pkQDUchjvOZAJ2O2+D5VMDDpAsgUUNAqsxC+8jKO1qxaQDyO5hLEzNxMqJaZV8lIIcx/UE0aYI/0aMDY3nSfCotg8FDXNEWGSD9SbHokjGtYyRoeR1+0mBgqelENGhS/P6GFw== 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=Qc37kZBNgc1Gr8OKw0BTKcwnNqHGtKJE6txXLZcjPR4=; b=qMsYUNaVMReC+3dg7i2pHjHRFEmmi+otJ2JiF0J4yDAaUu22ffhDk9fqbZUnVz/HuwdOdvCW5T9Df9A/MFw1d9Ngz9fu+YIgOz0q0PcL+1yqPPnVt2Uw2Y6YIpUvMB4eWX/KTuDCzhLzfUhx4/q3Sr+I+bS7KZlDQ6HYIC8w4X6F3lthPgCnGofZxV8ujMrYXHrvapX3dT90XYS/fPBEllDO5rgDA1GanJ+0pRuWTKNNFagVRhQvKZPl+BfFl2btnJUdAwBpIDbdRHFrfXFCh/DqL7aTahnMEzdkJvepahl83kvBXDIH4ULArCr9ZTV/MMvRBT/I7tO2ogWDWZNfUg== Received: from BN0P223MB0358.NAMP223.PROD.OUTLOOK.COM (2603:10b6:408:145::16) by BN0P223MB0279.NAMP223.PROD.OUTLOOK.COM (2603:10b6:408:146::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5273.16; Fri, 20 May 2022 18:03:33 +0000 Received: from BN0P223MB0358.NAMP223.PROD.OUTLOOK.COM ([fe80::9058:ff52:ab56:5b9d]) by BN0P223MB0358.NAMP223.PROD.OUTLOOK.COM ([fe80::9058:ff52:ab56:5b9d%9]) with mapi id 15.20.5273.017; Fri, 20 May 2022 18:03:33 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH v2 1/2] avutil/wchar_filename, file_open: Support long file names on Windows Thread-Index: AQHYaPywrYyA1nT6+EGODV6JkiNpta0h8QeQgAE8egCAAAMJoIAABhkQgATb9YCAAAFvwA== Date: Fri, 20 May 2022 18:03:33 +0000 Message-ID: References: In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-tmn: [Z8pKQf3NMfJQXq1xaoHoykCcY/4v6vBS] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 4b735666-74ab-4862-bcc6-08da3a8b0dca x-ms-traffictypediagnostic: BN0P223MB0279:EE_ x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: k1KbDfA3CJdFIXPQKVpjMNtqrB00GMT8c+KStSD8WHrvobehIMrQeIxFZp7HTq3Ard38TgMR7OAYi7guygrOs+k1I/LU4ZqEIWnumowxn9oEXGFKNI71TjIxGekTW1xmc7b09y6qMKTIYiJEhdciGD1lrzbgc3eteESsd2AdOMqIfU4cgXQ9EHAEGZmEV0RI/FX8BINKXb39M1oKsYsT/7kVXnF8O4le+f4vQHROlaOr3SqTj8059qkgRyHoiFsDOtC5H0fKpya6E6GIyf84ggtwFSD+JTTyb3lPbQLsdIwMynmQgyUm9LNkKmUYu0r995vpLbqxCReme7xBIYj9KOLJxkvmx5GoEXkNKNrIp7qxhGMpoY5VqFnJqoOAeMU6PbL7KaXhGGr8IwgKtXdbS4HeS3ZUqmqG+hL15W4n/7pHEYkOZ0wSdIy303ltPXOErlRJS27vDPy9rHbNlTjki/Srh2esncI+Df7qXGRHrWLH7PnDGuQXs1nf723mRx2K5htOoow1Cs4yBt0F6Eff7EzD01F9ajI+60rShGZyoEQsviA5k+g39D/YUIRfEYIB x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?dXdMTHdmWVdBSjRkRWhxSGlOUEZyMlM5ck1wNkFmSWxBaGFLd0dVWGg1NmFk?= =?utf-8?B?NkV6MjBUV3M0dVBMTjdMVVU5dVBkNHNiWm5LUWhyNFhMM2hYaWpZMHg1UGF0?= =?utf-8?B?TnIzek1EdTVQelA4NmJMc3RmT2tyaG1NWnJROTAvYkw1WEp3NGwvNG9xaGxs?= =?utf-8?B?ZG01U085Vnd5T1NxeGRhZ3V1MVpWU1hqeWRvb1VGeWtISndqSXk4RU1oTW4x?= =?utf-8?B?ZGNaQ3Yxa0pSYjUyVGJndWp1OEJXYkhncm1nYW50TmRxc3RHQWNBSFRZYmpx?= =?utf-8?B?QVYzTHNuaVlzOThrUlNQSmJzaUxaRFZXQVpRa0s2UEwxM1NhMjRlcHo2WmFy?= =?utf-8?B?ZkNsSUZHOG4zRnZxNDB3TFcvQ0ZmTVgxMkZuYU9yQUZhQ0hTMzBka3BKdUNU?= =?utf-8?B?NUNKY0d5UjkzYkY4YndoTHlNdEVmcjRJcldrdFRHRW0rMDliQmZQTmhuNFB0?= =?utf-8?B?MkJ6TXZnelZZbWxraVVQOUhaUlFpdUJWZkZzMGRUTllTYjhGTWN6Q1VxZ1Av?= =?utf-8?B?VFg2eDJkbWwzYzZLY25oQVUwUU9zUi80YWhFSjVXbjd0TlNyVFVMTFdHSnZq?= =?utf-8?B?RnhubTN6MWI1aHJZTmRuc1F4Q21xZ05mdHdMSExIbm9nRTVHVElITVNDTzky?= =?utf-8?B?QVQxSyt0OEdURnI4dWU3NkorRHRmVUVEL2hIYzgzVmZzK1BVVG5PK1VqLzJ4?= =?utf-8?B?ZlNBMlpvb1NubTVsdklvdUtyZWF4TlA1N0R1Z0E2SmxMeENESGgwUVZSYnAz?= =?utf-8?B?L3NTUUcvL0grUm1scjArRGV6dTdzOHpFUEYvZHFHbkx0UllBeGZHb1kxNkpx?= =?utf-8?B?M0RTOUJPRjR1ZUZ6ZXFhSnMxTGNXUkxlL0tMMmp2V0hZRFE1blYzd1l6MjZm?= =?utf-8?B?VkRXK05FZEY3WWF3cEp3bUwwVmRRSTBYZ1FDZ0h6aVhDTVgwSkVQR3VsTHhG?= =?utf-8?B?NldQc1RMUU42dU1meWJ4WVpUZHZ5NUZkMGpvS2pRQTJ1c21pQ3RUNXRBY2d2?= =?utf-8?B?ZEZJVzB4czRGdnJOSmlTdEt6d2ZlSVZHMlNoQU8vUmVtdUxJSDhWTTZNYXR2?= =?utf-8?B?QnhGT3pBU25ybUNEWWhWdlJpRW9tbk85YmF0WUxnQmV6d1ZFdmVTN21EYlJ4?= =?utf-8?B?N2d0VGtjRXZTbkk2Vk50Y3V4Q0RnazA4aGlnSldtNllVNHpQaW1oaDRqdUU3?= =?utf-8?B?WTNSREpqYTlrT0pwckJCWjNZbWxUVkpaSFJjbSswV3pTc3FVR2ZpUWtseFJL?= =?utf-8?B?OHV6cW1sbGNDY1FPUG15aXVHM3lFYmZXN3FtdmZXQXhKU0J4VFprVEtENWNz?= =?utf-8?B?MzJZellvNHd3WEhWMjFOOHBwZVlzZmhOUk1CVEpaRzlGb095K21HOXcrcDNw?= =?utf-8?B?R3dER2Q5dmFGbEJBeDVBVmtpc2JyOEJNRngxMmRUZTk2RmZISHRwWk9WTERB?= =?utf-8?B?dHMzOUhFSk41VW5oeGora3haaTdZNGo2YjdIVVVzUWNucTk3WDI3RGlDZENN?= =?utf-8?B?ZnBHNUQ0UW5JYSt3WHZiUHF3Z2o0Q0xJOHZRRFdxLzNNdG5TdlFnR0czOVNj?= =?utf-8?B?VFBtTjBHQVJKQUNUdWRGTEUzSGhOTGUvSC9QUGpDakdZbUFRdXh4QWJnRHdJ?= =?utf-8?B?QUEwWEJVTXBJM3VSUEVPMFh5TlVHU1dRa09GMDhpcHdDOXlWY2xCYXY3RlRS?= =?utf-8?B?V28rN1l5VkY3UTI3azFLb1M1eU5HNzJQT21XTWRWLzQzaXU2S2d5MThaaHpU?= =?utf-8?B?QzYwcGQxN1IzMHc2VDJRdEZpRjZ0VWJ3anZENUR2aUdXTit5ZThKRnMvVkZ5?= =?utf-8?B?RmxCYndrVUZsTDJhY2xXVHd3eENQbHFHZWdrNCtnQ3A2aVBDa3VqT2lNSVlH?= =?utf-8?B?RmNabEEzUi9JY25wdXRGOUZqOFJ3R0NOSU5McWNxRUx1SXpoc000bkVWT1Bk?= =?utf-8?Q?Em29MAcEVbI=3D?= MIME-Version: 1.0 X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-1ff67.templateTenant X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: BN0P223MB0358.NAMP223.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-Network-Message-Id: 4b735666-74ab-4862-bcc6-08da3a8b0dca X-MS-Exchange-CrossTenant-originalarrivaltime: 20 May 2022 18:03:33.2290 (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: BN0P223MB0279 Subject: Re: [FFmpeg-devel] [PATCH v2 1/2] avutil/wchar_filename, file_open: Support long file names on Windows 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: > -----Original Message----- > From: ffmpeg-devel On Behalf Of nil- > admirari@mailo.com > Sent: Friday, May 20, 2022 7:52 PM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH v2 1/2] avutil/wchar_filename, > file_open: Support long file names on Windows > > > Yes, that's true. But there are hundreds of other things someone could > > define which makes compilation fail. > > Doesn't mean that yet another such thing should be added by incorrectly > redefining structs already defined correctly by system headers. > > > Probably you didn't spot it. It's already there: > > # define stat win32_stat > > I'm actually wondering how does it even compile. All stat structs in code > become struct win32_stat, and all calls to stat become calls to > win32_stat, > which in turn wraps _wstati64. But _wstati64 does not accept struct > win32_stat*, > it accepts struct _stati64*. Content of these structs is probably > identical, but > it should not matter: C is typed nominally, not structurally. > > > I don't want to say that I'd consider this to be a great solution. > > But the problem is that the function and struct names are identical > > and when we want to re-define/map the function, we also need to > > provide a struct of that name because the macro-replacement can't > > work selectively. > > Doesn't mean that the should be named identically in FFmpeg code. > Naming a struct stat and a function avpriv_stat is a reasonable choice. > You can even define avpriv_stat with parameters the way fstat is defined: > > #ifdef _WIN32 > #define avpriv_stat(f,s) win32_stat((f), (s)) > #else > #define avpriv_stat(f,s) stat((f), (s)) > #endif I thought the purpose of all those re-mappings would be that plain Posix functions can still be used..? It's already the Posix declaration where the function name is the same as the structure name (stat). I'm unbiased. avpriv_stat() would be ok from my POV as well, but I'm not sure whether all others would agree when plain stat could no longer be used (without breaking long filename support)? Thanks, softworkz _______________________________________________ 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".