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 52649434B7 for ; Mon, 13 Jun 2022 22:01:44 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id AAD9D68B33F; Tue, 14 Jun 2022 01:01:42 +0300 (EEST) Received: from NAM12-MW2-obe.outbound.protection.outlook.com (mail-mw2nam12olkn2013.outbound.protection.outlook.com [40.92.23.13]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id D30D168B1C1 for ; Tue, 14 Jun 2022 01:01:35 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=MKLh6fo2ZdnJZp5TVhxFo96i8R56QMLMu8n1ddeOZ6p+7eHjOgCPDYue1hb/P6FWpteIo7eauDtDr3ZAzO4UWIfJkwcpEciqHL2NqNXp0ixyQjmDO5kz1nZBhoNzDUewSnx3wjm4gNKlWRDeWFEzIxu2Hia6bNR1zZjkNNxcaojZlsa24WstC2LkhmqLYkES5dr/G04IvZpKgr8x3Vy077XbkVogSAVMDIxjrkEfkNy4GiPeh7QQOiI7Zv1frFnr5vSwDqbiwc/2jSHJ93XDQyz0C05w7z5kn5/Degl5ZFKsLM3MGQEHkRswsPaGztjz0+CmVBkyNJkKYo3qU0i0rg== 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=3X/U77ogyYQqMjYfhbOtL5wr2UiK35aZn2uW5TWjIpM=; b=fbJYZkL7NoOHiXSzR1KTpcFhNJhxwhIFZzoA7w2+zuxTuJlyIDphJa5Ik9VNKHbZ8VbPkPMd0NpRLXrXNHfMjWC2R4dFFomo6/3opWRqa02zGzYnv6nYxl6wrlMNo4kVAAGXopiscQSszDw5aZLG6/jZ+wjKz5QoPTSShz05RtvSP6W4s5FdihADuIDWqcArpIO+JByWYoTg81Q6vw9pUQ0//+7LxFjwe/eE2i2wep8Eo8tgAFKBvhD64mc8hG2r5UlJcDmWkbbSMLyjKpJv84ISItxRU+VQe580AYuDAYTYZfm8AwDSKKjnSnNxH8b0lvrehJ5+FTiV4/xpyD1SWQ== 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=3X/U77ogyYQqMjYfhbOtL5wr2UiK35aZn2uW5TWjIpM=; b=htyZVL9CylmwxDrTmOtXaYEiIJ2uXCKCUdKEVb+kpoWYBwi/W5mTFO5kSu5MJQxv9e7XjsDV7rliZedRfwUGZuEJUe+BCvdORU3k0KZplg80U2a+qBC67m5OEYsJ1klPis//tGO0R5yQ3Ci3NVihcJU0ZPj/VetX5yyqozTR75dfCsEujAaUXEyQaGNPU42PoOVw2GSRaRMagUiiQucF/lbTFx0zp+jyk0HA5gLDIAHZaPwDYRS9eMG+SKdFWI6XmTRCUehL/24JkIgzHDzhhUUw+cnCSNerSgws+P0b+G0fT7RaVF/ulbpRdnJa1F1wQ+dDo/A6MkjN8222urxopA== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by CY5P223MB0492.NAMP223.PROD.OUTLOOK.COM (2603:10b6:930:24::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5332.20; Mon, 13 Jun 2022 22:01:33 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::f144:21f9:cdb1:5c8a]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::f144:21f9:cdb1:5c8a%9]) with mapi id 15.20.5332.021; Mon, 13 Jun 2022 22:01:32 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH v14 4/5] libavformat: Remove MAX_PATH limit and use UTF-8 version of getenv() Thread-Index: AQHYf0J4/HS9JaG/JUO4k628ixaVQ61NkYpwgAA/LwCAAAu7IIAABPpA Date: Mon, 13 Jun 2022 22:01:32 +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: [1eo+e66EGahj2fUTTzpZyLhLOXEgv35d] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 297e780d-b7aa-4b83-11bf-08da4d88470a x-ms-traffictypediagnostic: CY5P223MB0492:EE_ x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: G+2F+7eQYNtvPq1jdu2OKWk36ekG8veEd+kEXtOpsCBcy3YFvRupVp5WXPxoKx9PCaXzrRoYzPYYiQOxshEf4CoxlkdSAHxOYuYErlWBE0rCaEUnLlUlYbSEp32I0sDXuApQIbJbqln6gaTRA5quG5CGbCcDM+NOFAXax4ZbtvcX897oY1wqdYwf2ztzdUHdrlL9y3AWgtr0reEY7Y46JEtpsTiwizljGy9ddbUQJZDyNkxlVgrsoQ/NNe0z4Dbz5ekknFnf7v8YTjPRlAhtJN6Vjbis10kxzp0c6jO4SkzMdcuyTn3YarkD0FtlnSuPRLuhhcQmM1IhAxy0TXg4Ptpr9wJfQrqsaANzA82mlF6W9X2wBftLntX9BKDA7rertMkQ9NONG5IuPZymcy/4xOewYsu+KQ7m7ISXgfnPmvM/tOr1bNCw+nTZARTSSVZFVJFRVP8jlcJ+3UD5CE0+f0M6qtDHE9KZtnf6bWRA+AwfiK4gW9D874OQq8dGbXp7n2RONhZr5uyqf975J0y9XyFxNQtUbCVB0kpMp0tz73w5Gx2XQZFmzPnlr4o/eqnENiCIVrW6gFMn8FjxBnGXGga5c21vzFRQRaKYrww/APISqOYHoF4cxbipCdVzEGfJ x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?enZORjl3UWdhc1pnTytxUSt4RGF2MmYvSnEvNitGTldyMHRGY3dBK1Q4S1Za?= =?utf-8?B?R0lKK3R2NFFJVkx3cDc4YnEzUk50eVNidUN3TDRvZnN5Z3kvM0NxMlM0Q0d6?= =?utf-8?B?R2MyZGNCdkZ6dzIvYkJTdVByb0UwSGNzNzA2TXc3QUFJNlhWbWNDdUl2WGwv?= =?utf-8?B?bGxBbHAzNE1CckxkZFNpRVNwbkZZUE5KS0k1ZGN3YmlWYm9vT05hUnN3K1hJ?= =?utf-8?B?dGQ0Qm84Z0ZHNk9EMWxYZXRxL2s1a0dkcEl3aFlMdlBQNEVOc0lXNGh0Qmtp?= =?utf-8?B?a1cxTXdBcUlubEFlYTFlRGw5WW92WEErR3hHMDV1elkraHRzMW9YbzA4cGpr?= =?utf-8?B?YzBvcWMyUFdpclpoS0dtSEhENmorWGpyc2lpWGxGVVdlRjNqNWV6bitxd1R3?= =?utf-8?B?TzB2aUducU5tUmxYMVBoU3V6YVl3TkxWK2h4SVZrK0ZHOW1YS1IrNmtxUDJS?= =?utf-8?B?UHM2dEtUakxTc2JEcEZHYkNLNTFXME14bk9RU3Z3b1lMSVJiK0YzSFVSbkFV?= =?utf-8?B?R1ZHa2pyK3RJaUVDaGRrN2dHY1lHY1orTXI2aGVZdStrMVY0Smw5dno3RHNP?= =?utf-8?B?aDVTYmNKdi9OUWFkQjdWNWt2aE9VQzEvSTZNa3NhelVzVWYzRXdVMEN3Sisz?= =?utf-8?B?T2hnemZzNWc2K2FKTzZXeHZLYUhIVWhjQ05tRENkeVNoeStKVFFPbW96Y0Vu?= =?utf-8?B?dlBZVHYzTFBRUkwyTzNVMHVnaUdsZkpvWC83ekQyclMxanJKcUt3d3ZXeFFr?= =?utf-8?B?NjluKzlMS0w2Y0JPbmpLMkViSlpZazNWd1ZaeWIrUE45NnV3dHhUSWNNeTg1?= =?utf-8?B?NHNwTU5uNEhuU3NvZ0JxbmF4VG85akxsQUtTZXJDYkhXVjNBZmRoVnRCYnJv?= =?utf-8?B?MURFblVLWFdsZnlpNnQzZlhESlQ2S1N2SDM4NHo4dWJyZjBWb1NTU1ZFYWVq?= =?utf-8?B?Y2JubXRybGVNTjhwMUdmcVNOSktXMmNmYkxWdk82SHdqKzRRYllHZUJ0eFdM?= =?utf-8?B?UE9LZ0lONmpRMmtVK09EbjRvUFNzUkd0aS8vTk12ampWeGNVQzJ0RXhzcFdR?= =?utf-8?B?ZHoxTldaSFkzc0diYVdoSW5FS2V4L0pjTEY5dXJOSUpaK09iaVYyTW5tWDJ0?= =?utf-8?B?bFpMNHJ4ZkcyTFhiODlSdm1ETzlqcnlYb0llTElTemYxVGY1ZWpXYzFubkVa?= =?utf-8?B?RFZGeGYxWW1MaFpGUVorTjhIZWk3OGJNUTVtT0VZRXZ2NTBtZnV0YXBQWDZy?= =?utf-8?B?aHo5Qkp1OUZYU3VoejZWcW5HQ3dXQ3lsWHNkWGU5Z3JLeG1ZYjBYWVo2bUFl?= =?utf-8?B?dlR2TWxBK3RQdUJZdGRobkZMVWo0bDlVYnlxeDNkK2ZodUNUYWVvTjNqazVB?= =?utf-8?B?T3pHZjNwNnRMeFJCa0cySUNkV2pUWWdQN3RaWXBuamI1YUpSV2g0ZG45Q1N1?= =?utf-8?B?dE5HbXNBTTc5QUN3WldmbWlVei9Bc3RtYTdLYnhtQW93bzQxaE5udUIyZnVo?= =?utf-8?B?dW83RlJQMThhYm1iNkZMdHM2YWEvMDU4UXEyNFl4ZVZDZjZoWERMMDJVRklo?= =?utf-8?B?QTMxbnh3WmQ5VWpvdVFWeHlBQzNMczByL1hjRWo2WHlOWWhNQm5HUHBkcUFv?= =?utf-8?B?RUR1eStQOEhqV0hQR1hqMUovKzhjNEhqa0JTRVVnOVBCRUptS1l1Wnkvc3hk?= =?utf-8?B?Y2tHNWFZMmwyM3JjWERvb1c3bUpKQ3ZTSlI3aFd5YnZ5dkxKelRGMjlBWlZG?= =?utf-8?B?K0ViTWVXbnhyYWhmVjEwYnlJalQrdkRoZHZXMEVFWU9CdTBBK0tFZ2JTN0Ns?= =?utf-8?B?MzBZTjVSQzQvenlNTDFDWkllbXVaQnZpc0hpVm5QcHRUdU9YeFRFK2FqWm5q?= =?utf-8?B?RSt4bXg0elZueks0TWFzL3V1SEt5NnVRS2hlZVhxaVA3VTIxQWEwSnJNY2dH?= =?utf-8?Q?nEnL1FBFxjE=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: DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-Network-Message-Id: 297e780d-b7aa-4b83-11bf-08da4d88470a X-MS-Exchange-CrossTenant-originalarrivaltime: 13 Jun 2022 22:01:32.8611 (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: CY5P223MB0492 Subject: Re: [FFmpeg-devel] [PATCH v14 4/5] libavformat: Remove MAX_PATH limit and use UTF-8 version of getenv() 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 > Soft Works > Sent: Monday, June 13, 2022 11:53 PM > To: FFmpeg development discussions and patches devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH v14 4/5] libavformat: Remove > MAX_PATH limit and use UTF-8 version of getenv() > > > > > -----Original Message----- > > From: ffmpeg-devel On Behalf Of > > nil-admirari@mailo.com > > Sent: Monday, June 13, 2022 10:53 PM > > To: ffmpeg-devel@ffmpeg.org > > Subject: Re: [FFmpeg-devel] [PATCH v14 4/5] libavformat: Remove > > MAX_PATH limit and use UTF-8 version of getenv() > > > > > I like the version check. I don't know about all the derivatives > > > of AviSynth, but I assume you have checked that it's valid for > > > the common ones (or at least the original non-Plus variant)? > > > > Interface version was changed to 7 in 2020: > > > https://github.com/AviSynth/AviSynthPlus/commit/40900dc1c54c14ea9f188 > > c7242b88d464d067a44 > > three years after utf8 was implemented. If I'm not mistaken, there > is > > no way to check > > for a particular revision. > > > > > Two ideas came to my mind how this could be done better. > > > What's actually needed here is not a string conversion, we need > > > a valid and usable filename, and the function could be more > > > something like "get_ansi_filename()". > > > > > > The first thing that this function could do is to convert the > > > the filename to ANSI and right back to UTF-8, then compare the > > > UTF-8 result with the original UTF-8 string. When both are equal, > > > we know that the conversion is safe, otherwise we know that it > > > won't work. > > > > > > Then, we can use the win32 API GetShortFileName(). Which returns > > > file and directory names in 8.3 notation which (IIRC) contains > > > only letters which are valid in the ANSI code page. > > > > > > 8.3 file names do not always exist (depending on system config), > > > but it's always worth trying. > > > > > > Should both of these procedures fail, we could at least output > > > a useful message, explaining why it doesn't work. > > > > > > Let me know what you think. > > > > Too much work for something that was fixed on AviSynth+ side two > > years ago. > > I wasn't sure how important AviSynth is for you. Until I had given > the hint at the UTF8 option in AviSynth, it seemed to be a high > priority to enable the use of long paths with AviSynth, and what > I was thinking is that this might be of interest because paths > with non-ANSI chars are a way more frequent case than long paths > (on Windows). Before somebody will ask how I come to make such claims, here's why: As of Win 11, the Explorer still warns or even prevents you from creating long paths, but you are free to use any kind of non-ANSI characters for file and folders names. This does not only apply to the file management application, but includes the file open and save dialogs of all applications that are using the standard dialogs. 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".