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 0A17C40E75 for ; Wed, 11 May 2022 07:58:13 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id EFCBB68B369; Wed, 11 May 2022 10:58:10 +0300 (EEST) Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11olkn2058.outbound.protection.outlook.com [40.92.18.58]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 85B3C68B18A for ; Wed, 11 May 2022 10:58:04 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Er+HpuylPe2lSLW6tdnts5zyzWr+NQvtuSPXalknv0sveuKXvHtsbW4I0HrKDEsOhj/j0ncK/M1l31QL5nuKC5NuVNWjv8lryyzKbvw5vG4Yl54iy8vZP8thdcV4Hex/mDC7irEppX21pb35b9XnorgdrbjnYRUht5RkLCHPcQcu6ug4Lu9KFUtS0McOQU8Yi/r6JGsBcEx0tCCBBhuDUxS6qU0LxhadapkrAWfIC0Y9Yhn/sUVjRG1g3GRC3kZ7mwtQJWRGgTmNBk9rR4HWQpFieD+2h+bW2lIf4t/9xZM07pp2quC1T/li49Zv+8oxLJLzh9UsShdh4BgXg9eiqw== 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=jprciLmzt9yY5bp6tvTo4+6E14l4mMziJdJrIa3tTQs=; b=J0OCqwoxsmB8jj6jxlm0X2T6Jd1TgCIfTTrqy8tzKA3+ZZodAIIMF/cq9U3IkCDoy0ofGatGfJYbagBqnG4Ii+Y9jnMCMdlBsym61oVaJcOWTnoQ6ZOK7VQl9zS6iS/1GjCcyWjZv7N5F6OyIsEiD0l0+h23iME6++gioQQyM7xoNl81NoM/KjISylpiWXwv7mAf7pWKGA/mYpA3fN/W+trIxyVQFrQvPTG5qAix77bkGizDZBxMJrYUaS5rILXyISb/loN1pXFaYGUwYRaSsGOEv+Q9p5YYRRzePDaXfBA43CcUjFaWRlINp6Ai32BF6jVthROKzKxJ3B/xOCjRTg== 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=jprciLmzt9yY5bp6tvTo4+6E14l4mMziJdJrIa3tTQs=; b=YkjjFaLcyvH3t0oRYqf8z780pZsWHbEB0K/8VNtvPpq99DefbC9LIvKuiaIVLzajK8Xg30DCmMz/HVLN/4catEQYQCYHJb87DRHn8VctvRHOUF7VChLTuH7UF5uGR09Az39h48NPTW1BOz5IaUbkyi80D6kvqbRwd81zItTx8LYaK8VPXTPcFUuwHvveFlMDT9GQWmJkz0YBK06KU8vm6Sm7v1ACBW8q1wcLz4UZAyjaeokDmzFBW72soCbjmXhDeS2R1Yx+m2zGAMKCWrzZQukQydIsNHs8HhiP2V0QHjUHtn4FceeVNKUwJ/xscw9yTyfXD4cwk/CmNC8VQeztWg== Received: from BN0P223MB0358.NAMP223.PROD.OUTLOOK.COM (2603:10b6:408:145::16) by SJ1P223MB0410.NAMP223.PROD.OUTLOOK.COM (2603:10b6:a03:48b::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5227.20; Wed, 11 May 2022 07:58:00 +0000 Received: from BN0P223MB0358.NAMP223.PROD.OUTLOOK.COM ([fe80::ac80:535a:6742:2519]) by BN0P223MB0358.NAMP223.PROD.OUTLOOK.COM ([fe80::ac80:535a:6742:2519%3]) with mapi id 15.20.5227.023; Wed, 11 May 2022 07:58:00 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH v11 1/6] libavutil/wchar_filename.h: Add whcartoutf8, wchartoansi and utf8toansi Thread-Index: AQHYV9QNP/PRify4SkipW5U65/+zkqz/hzrQgADP4ACAAAFRMIAADCEAgAACGVCABt4dAIABGaBQgAhtCoCAABn5QIABMduAgACxSQCAAPjgAIAAApjggAT0jICAAAyyUIAAFw9QgACKfICAAAD8gA== Date: Wed, 11 May 2022 07:57:59 +0000 Message-ID: References: <40ff7ed7-3583-104f-4c51-91eeb9847d38@noa-archive.com> In-Reply-To: <40ff7ed7-3583-104f-4c51-91eeb9847d38@noa-archive.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-tmn: [O2bs0eRVsHm/5XVC+SLW+N/eJ4QuffEY] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 2bd60a67-dedf-47ed-a323-08da3323f7cf x-ms-traffictypediagnostic: SJ1P223MB0410:EE_ x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: VVR0yFgcx4ePD/awjFmfFG9xJGxHtt+z9qdyC0Glcyd0TQHGhPdatTARqcHkEtkdWpKqzX23OlmQZ2D/NRbaIdBrlUHeFngE75Vxm8cIpqc7yPtseuBU+T4LOzozdzKOnsCwFGXbAR7nn541E7gb8V5L9EJ21kfFDvDkKVm6gl1r+FCtcBhPymntQuMf5fp8QOeigrs7ccjHb0ZRsdfUeFzlhSaC7zI8mn3PyQJnBlikjvIMYY2g0TypUrQ3xLOy4h7v1awz1NUt5IKM98lHHy1+XkrhCmjvAJUBGfRAE+htswt5B/Lm/hx1uZd+qvT+IJnlkTVCQSdjztW56Gt/fMvTLeIQ41MRyuq8BKkHCeY32kseE0PnuGXoyD7BXDBFcsGC6iTGl0d98LTcwvGman9BENQ/2YwJKn/xZxBjpHPxEhlnKRwhdAqaVm/vmXz+8KEfGVzKr/0ThQFq9F6vAvj0B8DQsfT7cWme7vInzPRka51U950ORgBTHQhX2jaWyGBzMhhNtP25i3/FBscsiVu4Sh5DXQt/OnpUn3VrsCBAaiVmYChdtXUC9yXoS2Ds0SsSCGn0FoQK4xSqbfokzD1tp13mqexyuPgHP64I7APRiGXlCCVyt/neei9gwN/d3rPP5tD6mkR4CWr+5iBzSzMAPd8UTR7F+hm2XrcSZ+qd3AvQxYigXnn0aahtpwLonP5jn0qXVWjHkL6KYDOCbQ== x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?eGtKVStSeUVVdWJMVVhiUGtSVWJNYnNERURVTFh6ZUgyQkl3OVVxVUczNml5?= =?utf-8?B?dG50UmZyNGxGUXVLZDZjcW4xNFgyNGNQK1Y3OXVDdzcxcWhqcFNBOVo0OFEw?= =?utf-8?B?VlFSeXlaRVUyK1Q4U3l0OVM3TXRQVFdPVmZiN2pYRHBNdmJjVU51S2prSWV3?= =?utf-8?B?aGFTUmN4QkVwenU1VVZPNENpN3U0Z3pBK1RWZ3hIcEdZNHhZZlUyUDdROWRm?= =?utf-8?B?OGd3OW1NdUk1Z3BVL3lMRkNPUmVvcTkzaGFiWWlTRXk4bk5aMGZKdHBqRzU1?= =?utf-8?B?enJzUTlwQzZhZG5OZGZvY2tuWjBaRVd4WWwyODkzb2pPZnltWWxIeEM4cUJ2?= =?utf-8?B?eElTRkhBSmdralBkaUd1TElocWFwWnBBYjFvdmJYV09pNnEyVzNGTHlsUEE4?= =?utf-8?B?bzdUZUxmSnhqZFk2eCtGMXBqYzR1MXp2SHFpMko3NXlLTG1Tc1BuNmorVmdj?= =?utf-8?B?RklFRzlkU29PWUhpeHpCaStkcnZUcjQ0YndJS0dYWG4ydE9jek9ObWpWSFBI?= =?utf-8?B?TlJHYVJZZlV1S1FpMlcwaG55MjRXcWNGQXZmWE8xSXN1SzBUKysvWlQyRlpS?= =?utf-8?B?YmFSdktVMXp1bUowZk4zejZCNUJUZmhjNng5eEgzemRsRzlQNjgzOXVobEQ5?= =?utf-8?B?b0VNUUYvRVlCR0lUTVN6dzJtWHFZZHJrTnZoZlV3ZStBbkRhQTF5Z3didEM0?= =?utf-8?B?bk1pR2MwaWZZNkl3Sk0vMmxRZXZYUVBsUmlSd0wxNmNMQ05QbTlRQkxMSEFa?= =?utf-8?B?MHVqRlpVUmNiY3NTS3ZON1BhYWxIbktncXRNdEx3ZzhYOGJMTUcySFVJTE40?= =?utf-8?B?N3BXVkhvK1NjVTBJaGI3dXN6OHpFOFV3Q0h0NHpncHZZUjh5cmprUFlzL3RJ?= =?utf-8?B?aWdmTEFxSUJkeWgyVEszNXB4VzNmU0crU0ZRdjE3SXhEUVM4SEhscEhPL3Vy?= =?utf-8?B?anRtV3BpZ09TdjMxT3E2dmlCQ3dobzhudVA3VlgxYmdhNHpDRUNHRE5DL0hy?= =?utf-8?B?Znc2MjQ1dUp3ZkxIVGxWd2lTU1ZGUGd1NTl4NEpyT0x2Uk1pczNRbDdiR0Vj?= =?utf-8?B?SzVPRnhMdk5iNTFLSXYyM2I4MnZEQmVhWnZkL1M4cFFWYlhiUW12VGtsOTJL?= =?utf-8?B?Zmd6SEQ0UGtUV1VvYXZDRkIyZXdGOVdWcFA5MnBGK2VLbGlNdTZNNjVtcHh1?= =?utf-8?B?THNZUmVLYUhRRzViYlZLVGM5Z2dROFlrbWU4dVhtM3hLbW54bXAwV2NuTFF4?= =?utf-8?B?Y2ZKc1pnS3hGNVY2YXBMdnJUVjdvQngrdElLaFJsNW43WDFVdWw3UTk4aEY2?= =?utf-8?B?Vko2dXhzMDB5OG5qTXJ6MERKbE4rRHgwVERQZDBJZVdXWTA4VEpVMmtaTHlp?= =?utf-8?B?Y1FMRHJJOXNjTkVaSmQrVHB3NCtkRVR4RHpJMi9keE5FMndZQW9kdnhiVDdv?= =?utf-8?B?VUtRbFk0bnkvTFhFZDczMGVGSVpWdTdKUXFDclNZQ0c4bHl6cW9PSG9iNnA1?= =?utf-8?B?SldEcVhYNG1LRmF1VU5NSkM0b0dJL1l1MjluWTlOL0tGLzhidVhiaE1ZZUhV?= =?utf-8?B?QlY4ZGNmcDY5Q1pEb210bU9pdVk3b21RakNCelZzaTJJU1hmYUFOYmtIcHEx?= =?utf-8?B?ekd6enlxZklEeWFJN1MwRjB4ZytLVkp0MExLRUdQbU5hcWI3TGhzWVhmNmdI?= =?utf-8?B?cUswY25FRTI5TWZpN0JSWnNyYkxDRE5PMGIvNnFxbnhYRnoyN3EvV2pxTnV4?= =?utf-8?B?TjVwQXIrR2JZRUx4S3FpeUpIK2ozRDg3N0ZqTngzYWVPZ0ZnbWlNeHJFUzM4?= =?utf-8?B?cFYxVGV5K3hhWThGejBBdm8yNUNpZ0NBSGljNWF6L3B3c29TL2g3U1AyOWFh?= =?utf-8?B?SjliVkpZam9lem55OG5hc0lqUGNPcVN3Z2xqSXFNRVN0aDZjTjZ6THBRbjVZ?= =?utf-8?Q?argB/LYTGcw=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: 2bd60a67-dedf-47ed-a323-08da3323f7cf X-MS-Exchange-CrossTenant-originalarrivaltime: 11 May 2022 07:58:00.0330 (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: SJ1P223MB0410 Subject: Re: [FFmpeg-devel] [PATCH v11 1/6] libavutil/wchar_filename.h: Add whcartoutf8, wchartoansi and utf8toansi 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 > Tobias Rapp > Sent: Wednesday, May 11, 2022 9:46 AM > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH v11 1/6] > libavutil/wchar_filename.h: Add whcartoutf8, wchartoansi and > utf8toansi > > On 11/05/2022 01:32, Soft Works wrote: > >> > >> [...] > >> > >> The prefixing can be implemented as a function and then be used > >> in file_open.c. > >> > >> Other file system related functions like mkdir, rename, rmdir, etc. > >> are already intercepted in os_support.h, and the prefixing can be > >> applied there. > >> > >> Maybe I missed some cases, I have not fully analyzed the situation, > >> but surely there are just a small number of places that need to > >> be changed and not 587. > >> > >> > >> For the procedure of prefixing I would take a look at how it's done > >> in .NET. This is probably the most mature code for handling this > >> and might save us from dealing with issues and regressions until we > >> got it right. > > > > The logic they use is here: > > > > > https://github.com/dotnet/runtime/blob/main/src/libraries/System.Priva > te.CoreLib/src/System/IO/PathHelper.Windows.cs > > > > Probably it can be simplified a bit. > > Out of curiosity I searched for some automatic path prefixing code and > the author of this file claims that it should be handling most corner > cases: > > https://github.com/JFLarvoire/SysToolsLib/blob/master/C/MsvcLibX/src/m > b2wpath.c > > That amount of logic inside CorrectWidePath() does not look appealing > to > me. And even if we simplify that now I guess it will grow once the > corner cases drop in via bug reports. I would follow the MS code, that will be the safest way. > So I'm in favor of removing the MAX_PATH limit, converting needed > Win32 > function calls from ANSI to WideChar, and adding the longPathAware > manifest flag. I'm not sure how much you had followed, so please excuse in case you had already read it: the manifest approach does not work on a default Windows installation. To activate long path support, the users needs to opt-in to a behavior that is probably deactivated by default for some reason. Also it requires administrative privileges to make this change. For me - and probably for others as well - that approach is useless, as it would be the same as if there would be no long path support. (when you cannot rely on that feature being always working, you cannot use it) Kind regards, 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".