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 DAEC340A2E for ; Wed, 2 Feb 2022 22:19:05 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id F411268B0E7; Thu, 3 Feb 2022 00:19:02 +0200 (EET) Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11olkn2014.outbound.protection.outlook.com [40.92.18.14]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 875C768AE99 for ; Thu, 3 Feb 2022 00:18:56 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=aGQmd8sOd7dcmqWjQc7cSHFAWBv7k3UJt1iFFpsxZsuNTPIV9+TgMEXh5oGBGIAWVJVNEgvOkb4kmCXxhU/rabKXG8D9YLExeRUT5/NZC06p1AnfIBdCp1zJBxbjR/TmtdaYdZss8sNPjcCvCZaSYgBX/SKgE+MaapQifOrtusaIuIyEpTAQFEAnSB2nx2l+LJ8xnEzQ6kzLB+nwA5QEcYXYpjittktmgngmcKVbikwDV8jmcQs4hLXhEQUOPgtHTAH2dOdw6GUA0v8nUdZRpOYssCJqHEbE7jpctH5MwZTBivMOCL5/V5hAUZJuqSK1IZiZpU5oEHFbICRQMBT1Qw== 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=gNcKtgKE1V/QiydA7FUOY2pPBP7FduQpGx6XJGPvZwM=; b=egEGCLbhDlWt+xyPTJjxLQfihG01IiEKWICB2PKDNFEBkI2hR0NhidU0vTG7kwV7JgduAEXba/FN4CgH96CGVLTdRAZ4gR1ZNf1KV/OBg49xw2OmAa2UZOtgXfhswt5G/Qtc9srEoe6ty/vRgwTuF8xtpjbUVA31bJpO/hG/I7fvbXjvjImeBNHfTbi0oR+bnjOdK5K8UggGZznF/0LXBhfHzxIbFdFoWGxb7phmE9FSY4x6+SIdotNgtpOGMTlDeVMzRRA2g58Y/jYn19TCVfPKIEsmsJ/CvEYyqsd2RoiOPP50P62uiOM/uy3QXr2K9pfE0y2AuJc3eNJS9gcyVg== 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=gNcKtgKE1V/QiydA7FUOY2pPBP7FduQpGx6XJGPvZwM=; b=bxaZkFg5JyoqsjKDz2veql8tqb0pSoStGrDnpf3L2CUiYhEp0jQN4PNHDbVfgiOLSEGG2gl0a7ZUYVlVVOz/AzJLHpD2A3hFzMNmGDt0nGuDX5l3aJvQbtuFEDASNiDDiMflZCedlPUZSkjg5Lkt91re0CF9+GtMWi9vlrkbBN42RBNaPDO2ybndJnGjuI6mpCF7pPSYIVExe72LbRvkQNaDDnpC0WIylRBczXXGm3W++GfQPgxh4miX4qv4c2y2OvBleQ859W/7+TfHQBWtirsUXroftBzMrtfLLhjMRWrBwkHHsLjiPdrYARfK+dS0CUVO3bc9V0DDaVMbFYqyNg== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by DM8P223MB0287.NAMP223.PROD.OUTLOOK.COM (2603:10b6:5:317::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4951.11; Wed, 2 Feb 2022 22:18:50 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::496c:9ebb:f23e:8e6b]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::496c:9ebb:f23e:8e6b%4]) with mapi id 15.20.4930.022; Wed, 2 Feb 2022 22:18:50 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH 1/2] avcodec/{ass, webvttdec}: fix handling of backslashes Thread-Index: AQHYCwVRgXvA06Jx3E6UTvILqno24ax/DxUAgAAiirCAAAbTAIAAAGXwgAA3OwCAAEDQIIAA5q2AgAAExGA= Date: Wed, 2 Feb 2022 22:18:50 +0000 Message-ID: References: <20220116181655.6407-1-oneric@oneric.de> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-tmn: [E+GDkNV8dDJBqJYxrZ7H0SIoAv951tY3] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 12cc1a11-b15b-4d9d-e3d1-08d9e699fd89 x-ms-traffictypediagnostic: DM8P223MB0287:EE_ x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: YXY2O5iKBqMz3ubTFwpWLPkBpJGHTUk+fufJ3OpSCmO5gezI/vHYVCfgJfA6oHmwiC/DNWdL+7fKUOg/RbTyNDvaPzbAf1h+jyn/z7/4utr2IfxqxvGoczKn/iItv5cMBM0QUoDjqRBqzN3TuMSx61BZsLK0TJ1ys9o+70yCsdY/vGg9RwqulMdgiXkso4nA/w4PIm2/rQSyc2LHOo1BiI7dV4pl/7TfCWL85ccXt0iKDk/UsPaPTcC/f6QiHejmodT03fd5YZdO93xpA609ryReP5QXNyA0gjXE7bD8gCT7+pJhCBcbd7p2325qeTI8Vi7YKt6XI1XdmPlqKG69v2/P++qVTColOBC2OE2LLkPsSl+hQWT1cUCMC8IDUIAMgLBlaPdLBBNmgHirDKWAem4MAyxX4eU85Gqgnmct/2j7/D5GUDHCSkiZgQu5uPBcn2dt0xY1i6H/W4hYLCeQEa7NOAx1dtSGWlDnVlU6hrMbMCmLHn2X8l/4G3xUxfQtyPovgmEjC6C7SKe+GlR3LzTnxSMKuO4W75J5JN7X1tyQjmz0eEzhixcLuDw1acSH2eAULK5AlStw3GfKnzvBlg== x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?bVZYNHI3U25OekpvdHZTUUVaYU83QW1LVTRtWGxsYTdkeTBRMGFJcGFMb0Q5?= =?utf-8?B?L2xwaDUzVXJvYks2U256MGh1Z1JWVXljcHZrZGJPVThQS3J5YVdHVlpYdEZZ?= =?utf-8?B?OVJENElXV3RVUWhSOFl3ZEtOY0Exd2RQOWZlZnhkMnJMVE9CR2JSSnAwSU5m?= =?utf-8?B?UFBQaXlqelJmdDc0UXVsRTZJQ1RmOWZwMEZINGw5eUl1RWlkR0VBN0s0WWE1?= =?utf-8?B?Q2M4REJ5Vklnb2h5TUNuMUViRmtJMkRhV3ozTG5YNVlFa21pYS9xandSQi9D?= =?utf-8?B?UlNsWUNwd0IyT0JqTUwyc2hDdm5WbEg3Z0FFKzFvYlhPRDdjd0VJZ1hJSXZB?= =?utf-8?B?bTN1eThCbTdkSkM3dSs5cEhUUlBhMHBYQW9DV3lNRk9NUVdiRGhWNjdpdDZr?= =?utf-8?B?dXlzUzMzU0dLbkMzSThmb05SclpUWksvSUtZV1pMcWV3L2VueU5PK1lhZlQ4?= =?utf-8?B?WlNmZ05RcmRHeFVSOFJwNjVwTFJ2S05VdCtSaFZOVVVvdUR6cVdHUXdSSTRi?= =?utf-8?B?UURyWnJRdk5wNDJ4bThuRGFkNWV3eVBld0NOMVdLd25sQWpDZTdoK3kvMXJY?= =?utf-8?B?eGJqczNKbkxWQW91ejZVQWxFZm50WGhUT0F6dUpEVFNWWkxEUkMxcVBWSGZr?= =?utf-8?B?QktJYnh6T2FTclhoNVJ3V2gyV21HaGJBa3RLRzdJbS9KbDVBYjIzKy9hMFdl?= =?utf-8?B?VUF6d0dKVXZSTkZvcE5DVGpLLzNMekR3VTRNZzFoSEUySDByVDUzeVIyTy84?= =?utf-8?B?RWZnamhHVFlyWFJlUEVqNzNJZWZGbVRYQ1FYYjVFMDArMXBiSG9zUnJaRjRk?= =?utf-8?B?dStXcXZKc0dTRTRlWFBBbEdnRE51bEFMYS9DZTNONnV2YjdXdGVtbDl2UmhR?= =?utf-8?B?OUhUc21WaFdldVZyTGFLc29Dam56NGRjVm4wN1piTDVLaG1PVysxMEl6aXM1?= =?utf-8?B?QklwYlpQL3YyU3dGQ1hKVjJuQlZoNXZwNzRJUVBObmwrbDkvcVJ4VWZ0d1Zr?= =?utf-8?B?ZzdpS3M1NmFMK3NCMVFwdWduaTRSVHFtRS96VkdLeG9wTUpZWFpQYytIV0JS?= =?utf-8?B?eHNIVjdWTXZ6aGZOVjJHZW5acXM4b3hqV05BakwzdWtTZFJHNzgvMnpoNGZ4?= =?utf-8?B?YWF4ZkpGUml4VXFNd2dYeURsbDh5Nkd3aWdIU09EWDFsZlBtOHdEbHAwSWln?= =?utf-8?B?YUdWZ1JCZStab3pMaEV2VWNHbThTRkRxQVBKcDZncFNWalVrUGN6SkdtQWxR?= =?utf-8?B?NnNQNVlqeFhieXhESnErV3VENm5DdkpweS9raCtkU2lmTklKUT09?= 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: 12cc1a11-b15b-4d9d-e3d1-08d9e699fd89 X-MS-Exchange-CrossTenant-originalarrivaltime: 02 Feb 2022 22:18:50.7224 (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: DM8P223MB0287 Subject: Re: [FFmpeg-devel] [PATCH 1/2] avcodec/{ass, webvttdec}: fix handling of backslashes 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 > Oneric > Sent: Wednesday, February 2, 2022 6:03 PM > To: FFmpeg development discussions and patches devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH 1/2] avcodec/{ass, webvttdec}: fix > handling of backslashes > > It appears my previous mail has been interpreted as some sort of > attack > against you; this is not the case. I am sorry for creating such a > misunderstanding. Don't know who said that, but I don't feel attacked. It's about technical details and opinions about it. All good. > The mail was merely meant to clear up the misconception that it would > be > impossible to send mixed line-endings via mail due to some SMTP > property > using some illustrative examples from the current thread. > (I have no idea how SMTP works in detail) Where did you see such misconception? I think almost everybody knows that this is doable over SMTP in some way, and there are in fact many ways to do this, even when it would be as stupid like attaching as zip archive. It's never been a matter of SMTP. It's the GIT tooling in the context of "patch-over-SMTP" transport and the way it is using plain-ascii-text e-mails for this purpose. IMO, this is a design flaw or just a bug when git-send-email and format-patch are creating output with transfer-encodings 7bit/8bit but mixed EOLs. I suppose this is due to the tendency of GIT to consider text file basically as files with lines of text where the line endings are exchangeable and rather a platform detail than part of the text file's content. And text files with mixed line endings are an accidental side effect of multi-platform work and the endings can be re-unified at any time. That's also the reason why format-patch cannot warn, because bazillions of code files exist which have mixed EOLs (accidentally and exchangeable). Yes, yes yes - don't need to respond that GIT can handle those cases flawlessly and precisely when configured appropriately (and when not using the e-mail features). That goes without saying. I'm just speculating about history, why it might have happened that it doesn't work flawlessly when using e-mail. Via e-mail, it only works when using base64 encoding like you said, but this had only been added at a later time and wasn't part of the original functionality. Further - I could find no evidence for your claims that base64 encoding would be chosen automatically depending on the content, neither is it on by default. There's an 'auto' mode which chooses between 7bit, 8bit and quoted-printable, but not base64, which always needs to be specified explicitly (or set in .gitconfig). This is the way it is documented and I'm seeing exactly that documented behavior on both Linux and Windows when testing with a mocked smtp server. You had contradicted my statement that it would be desirable that format-patch would be able to do base64 encoding as well, saying that this would be the task of a mail-client. This is not necessarily the case, though. format-patch can generate patch e-mails with multipart-mime content, where the patch content is created as an attachment mime part. format-patch does that with a transfer-encoding of 8bit, but it can't do it with base64 encoding. (I'm stripping the rest of your reply as it is all covered above already) 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".