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 ESMTPS id 4236C4C923 for ; Mon, 10 Feb 2025 01:37:10 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 27CA368BD4B; Mon, 10 Feb 2025 03:37:07 +0200 (EET) Received: from NAM10-DM6-obe.outbound.protection.outlook.com (mail-dm6nam10olkn2060.outbound.protection.outlook.com [40.92.41.60]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 9AF7568B65B for ; Mon, 10 Feb 2025 03:37:00 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=Srmyqg7bW1vbSLAqncj3ATSSeBlqXdxueud8o9p1updvnyuUjWn/X4C8ncIpnKH9qimthQQLJsI3NDf9arnJs+X+mO9tcyZsnwxFAQenC23PIYShZFe4lrcxsD7M8JR0EnfKY6W50Ur1aZ718JIGNR1WGxA2e2wznKvB3M5dW+TMHlPfupS/44s7S6RSsglWtkMSedgaIhxH2HStA7SiPFBMlk5QQiFm6SdVxzJFcdyakYwyF7OCfBmWdzig0/hAnnDg568Ey1M1JfFrjKJbDojdo2eY/bQ/XW5PN7y5ylE+nWmxjJIiN2133RiwCf829RgWjXLQMHq1IvErNqGeYw== 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=lp7Guq5tQtxg0s/u4kdRY5gM4xkTM4y2/IoZNHGprtA=; b=yuKlpuZE6Ojy4yL4ew52r7356pksaYhHwToExxPsRavyHFk6btz7puGfDR8i8mJsEWUDRbAJ+TlfMeHx6OD3CAvPtob28Sh5YfhDj71VjLQwx3mN6v2lMmCpXk0/UOh9DrRdQzczacA1BUhFpU4S10P4l8ed1izvJTXdyiFh5OiFJHnS0PFcpaZlfNUY6qVc8FUUqZ0T91C9q/vw67CTefvPBJLfzeWmQ33e12sLdqU5/BJzlpITf4LCiXa31LfphaRd0Gx0Xs7qBHDHiznNCwqucCo11ij/P+Frzmjq4Xk4LoYOZWU1Gufv3eHkcYiNZT58QFk3vSLlhfzNM5NeBA== 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=lp7Guq5tQtxg0s/u4kdRY5gM4xkTM4y2/IoZNHGprtA=; b=dshTkYKf6GfiTX5oV7qS3piXRm+R9rnGbnLBKDBt0QfkdOVI0exvHOamHwH5fJd+M7Ihhjcbq0fosFfVIUacWAYLMxeXVLWvUtZiFbYPVBAUl9Ji134EoVVrnKvG3yjureOFkiUpV57IT0N2RYuJPSLLnZGh+oYL4wJji/6VQVeGmDzWku11ZQ4JJyyrt4x1CyiTiWlFUW3e3MsYD4AnclCkJf2fY9/YWkRLXnGKnKHFldWBUv5tLn52DyB2AWHvsFsm7vqb/+mwnmScfZO6DXtlzDvNvhuGL8Hj+Nq4t3g9iGE4Bq0VlGoSybYHsAUzQHy5tnAP6e1ZZtg+EN5K6A== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by CH0P223MB0155.NAMP223.PROD.OUTLOOK.COM (2603:10b6:610:e9::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8422.18; Mon, 10 Feb 2025 01:36:57 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::bf09:8e9:b07f:98a7]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::bf09:8e9:b07f:98a7%6]) with mapi id 15.20.8422.015; Mon, 10 Feb 2025 01:36:57 +0000 From: Soft Works To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH] avformat/hls: fix typo There is an extra space in the original comment Thread-Index: AQHbe1Ouyt2tf1hZD061EvTxa/em2bM/tGHQgAAGxICAAAEmYA== Date: Mon, 10 Feb 2025 01:36:57 +0000 Message-ID: References: <9AF64C0B-FBE2-4F93-83C4-040AFED59073@gmail.com> In-Reply-To: <9AF64C0B-FBE2-4F93-83C4-040AFED59073@gmail.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DM8P223MB0365:EE_|CH0P223MB0155:EE_ x-ms-office365-filtering-correlation-id: 9f1e643e-d42d-4421-1397-08dd49736853 x-microsoft-antispam: BCL:0; ARA:14566002|7092599003|461199028|8062599003|15080799006|19110799003|8060799006|56899033|102099032|440099028|3412199025|41001999003; x-microsoft-antispam-message-info: =?utf-8?B?dXhNNENUUU5UOUsvWWdjVVovWGJsSXpkbDArVHFaRnZPU1lVWnFpZUZKcFdK?= =?utf-8?B?d2tBVXQrOHdsa1EzVVp3ZXJpZjdCbVkwS2RLcFN1VmhsQUJQUk9MWUNzcm9N?= =?utf-8?B?Y2h0ZmNPeXNEMDMxaU9IQ0VmK0xkbTI0WnlBUk8vQXEybndlQmUvengvdE15?= =?utf-8?B?RGRlRW1OUERicmhmZTA0eGdNdFJjQlJDSjhNeDJaUklaOXY2b2NhcmpEQ1dG?= =?utf-8?B?QmFMNURoOTlvSUJyS09BNzZzUWpJMVJMem9hZ2VLTXBiTDZDM0pWL1V3UWlG?= =?utf-8?B?Yks3UFZWbnU1UlZhT0tYOW94QnhJT3JIbkRDbUNMbmtqNHFlTGdCSTZmcTE2?= =?utf-8?B?Y1BzUWFyNkZodkZrd3ZIbW0waVYzeFdDaXczQXM1N1c5R1N0blAvT24vZVBC?= =?utf-8?B?ZFJqUFpsR2MxQ3Z5T3I1VVhRbFhBWE9oUDJ4cjZKOHJaN0RSTHhmUFI1TWFv?= =?utf-8?B?YW9ybXRFeDdHcHpOZ1p0MjRxTm5SL3JPT3I5TVdHZHZ3K2E2OU9iNUZJeW11?= =?utf-8?B?NGVhNWlTekNSNEpuUDAzQ2JtczVIaXJ1dm1rSG95QXE2UnQxNGtuR203UW4r?= =?utf-8?B?M3A0R0JOSm9FMm52SUlsOUVMWU9nZ0lNaFJkZisrY1VzcUZSS3RHTFBJTEZF?= =?utf-8?B?R2doRG1pQVVPb1ZVbVZOcm9UbkZRUzh0V0R5RFk1eVlJUnkzZFVKdWQvUURo?= =?utf-8?B?ME13aW9IMlFUK3REUXZwQjd5WmJPenYrUGFGcEo5dGVRWCtHbmJXNk1YQ1l2?= =?utf-8?B?djhwbjVVOUxndldQR1RNTWQyVlNvL1JKTEQrSStERWY2VFNMYm83TU4rTDlQ?= =?utf-8?B?WGhtTE1XaERvTEMwTlNYZFdFaGd4SWFFTTU2MGQrdWYrcWhLMFhzWjNZOEc1?= =?utf-8?B?VEEwMmpiWTVLcFBmRG1GM1FidXhwTDBmOTNaVXZvUmsyWGVGeTJTWTZ2eEJv?= =?utf-8?B?Z3BYRlFuekNFcG0vQzdtVEV5VSszU1Z2c2cyL3pGLzJLc042RmlZUzVuWk1D?= =?utf-8?B?QW1DbmtpUlAvTGhINlF6bG5Qd2xFdkh5QkZ3RjlRSDdlSm1UZmZ3SWVnYmU4?= =?utf-8?B?MFExYitWaW1pcXo0amdRWDVId3Q0dXVMNUt4ODgrUGsxTi9TSDk2VFVka3g4?= =?utf-8?B?K1duc3FsbXFUTEtUREhXT2hNeTB4eUs5MkVaS05yZjVDaU4yTDM2N1Z4NU1C?= =?utf-8?B?VU1SQlh2MXBJVUhkNCtFbVFDL3pMdTc2ME1oOG1OTkVPTytjOXMyQ0FRWEEv?= =?utf-8?B?dXJwMHphMXJtdVlDRlJ6Zk55Z2IxVkNRNSszTjBGN29UUVVDWFpnNml6R0dN?= =?utf-8?B?dnpiaUlnQmJIbnVlZEFMbStYZktiQ1JEVGphM2cxcGo5c2ZGUCsrQVczcGt2?= =?utf-8?B?QVJib0M0MG9tc2h5bGFDeFd6N1U5Z1VyUWtHTFI3SEIwaGRieGYzYk8wSVBQ?= =?utf-8?B?ZzE1dHJ1S3RMaDNkQ3oxN3BMSTZwbFAwclFnT2lpU2xzcUJJY1pVSHMzVFA5?= =?utf-8?Q?697tZA=3D?= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?S0x3NngvQWVhRlovcW1XRVhBUitVV3FVUDNQSkhuOVgrcmFQRDkza20rQzYx?= =?utf-8?B?SGxqbTJPcHFROW5ldklYUlkwc1lHcjNZeXhZaXRHQ2hQWUR2ank1WHcxM2Fv?= =?utf-8?B?U3U2V0wvekd1UUNxejRmbVVPbWF1S0xOYW0xVldIeHdBSVJVaHRaRWtva0RN?= =?utf-8?B?Yks5Vm1ibnUvQUlWcnc5eGd6L2pNZFNTbXdKc1dSelcwVHpMaktIRnMrWHM3?= =?utf-8?B?Y25JYkhqWXlqY3B0cnVsS0dnRVhyeHVLUDJZQnZnajd6WkxpbEV6MGJBVXJ2?= =?utf-8?B?aUFzM0laaE1jL1hhUTZqWEJxR3labFlRakFtMkFFUXVmcjFpVjhPM2tXMmNB?= =?utf-8?B?OWNtb1FWWHRoOEY5Rm44djFMVitPbitEZ1hVa09yZjJydERpRjh0dkFHSElw?= =?utf-8?B?NEp1VGhGdDJHcGk5YmhjYmNuaHYwNVZFeDI5SDlyMjU2TDhYdGFrVG9CU3dF?= =?utf-8?B?ZG5YWXFJU2p4OVRaTmlZTGNGUDVyOTlnZzNqS3dXWUpWSXR3b1V2VVFMZElq?= =?utf-8?B?eWlCQXBoM0dPVzk5cUtQWWNpdlhvaEF2MkJ4Q0pjZVdJeS9oaDlGa0R4R2k2?= =?utf-8?B?cWZ4Z0N6TGliY1dDbFEza1hSMkdxRjFpK2pWT1dtamptNWdrS1lKZk51RW0x?= =?utf-8?B?SWcvTkVLc09ucnJ6ZjdvY1Q5YXNIeGIwSWtNM1YzSUloWE9ISkpGeExGNC9x?= =?utf-8?B?WjY1NGQ1ckM4aENXMGU3UEpoaHBDS0hXLzFkcTU5a0xuM2RkZ2Z6L1BRU2dE?= =?utf-8?B?TGR1bmtMSGMzeENBbUxQeWg0UUowcjk0cGgzTWNZN1ZJTGRreVdtTnk4Rm5V?= =?utf-8?B?T1FFa1NoOUkvcTRXZkZvM2tQNWZmMDduMVBDZkNqcnk3RVlkRnM5SW5FTVRR?= =?utf-8?B?dElSMkc3V01CNDQ3c0RQZitNTjk5dUhaTnZib1p1Wk02c0pKczE3aG9wR1F0?= =?utf-8?B?bVNWdkR6dEpKcllodmMvU3ZpWDRWZVF3VEFiT2lhaEtoczY2eVRsckJwVEM3?= =?utf-8?B?NFlGcmxOS3h3R2Z6Wkw3Njg5ei82eGl0cFFDS3lZdEg4TlViK1dwMFM1MUtx?= =?utf-8?B?ZDluQlZTbTRITmlnY1lnSnJiZjJsMy91cXJ5bE0vU0ZXTzdYOG9PMTBFaXpx?= =?utf-8?B?bjMxdVhFUm1McGRFT2NQWEpsejl6YUxsOEpsWVNxS0xqbldFUTRpdmwrdDJB?= =?utf-8?B?anc3OUN5YjVTR3FROXR2dzFocjROakJqTVlNcCthc2l0MDlic2w2TDBvZ3dN?= =?utf-8?B?QWJDQnFoN3Q5MVFsbWRrK3ZNa01rYWtvMkRWM2tOb1hmRngyMEFObnBzM2Y0?= =?utf-8?B?NlhHaWQ3WTQ3TGpneXpJNWtJdXJ2TWJhUUpkWjFVQ1hpMXphYzVhSXJNeW52?= =?utf-8?B?K2VQdHJYOU9mUFFrTWZ6Z2E1RWQyTmNHdUxtMG05alpHUTdQeStOZk1kb21v?= =?utf-8?B?QTBmais2cUhlOGFVcEg5RTlXTXV5cmZCUGpKcHQweTdXY2JOeit2RVBnYU95?= =?utf-8?B?b1A0NzI3WWxsN3cyQlZxNmFod1pUY29RZTE3MTVFOHhTeEJQcjE3aWlwcDFi?= =?utf-8?B?MzhXNjNTQndSaUt4Zk11K2Q5ZElCRVlQVzJBMGQrVUdab1RmaEtPWmpBT09L?= =?utf-8?Q?ZKyBsTVVMOmm/KCjSIstceJn8if+qG4cOkaVNLD5Gw84=3D?= MIME-Version: 1.0 X-OriginatorOrg: sct-15-20-7719-20-msonline-outlook-92255.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: 9f1e643e-d42d-4421-1397-08dd49736853 X-MS-Exchange-CrossTenant-originalarrivaltime: 10 Feb 2025 01:36:57.6676 (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: CH0P223MB0155 Subject: Re: [FFmpeg-devel] [PATCH] avformat/hls: fix typo There is an extra space in the original comment 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 > Jack Lau > Sent: Monday, February 10, 2025 2:13 AM > To: FFmpeg development discussions and patches devel@ffmpeg.org> > Subject: Re: [FFmpeg-devel] [PATCH] avformat/hls: fix typo There is > an extra space in the original comment > > > On Feb 10, 2025, at 08:54, Soft Works hotmail.com@ffmpeg.org> wrote: > > > > > > > >> -----Original Message----- > >> From: ffmpeg-devel > On Behalf Of > >> Jack Lau via ffmpeg-devel > >> Sent: Monday, February 10, 2025 1:34 AM > >> To: ffmpeg-devel@ffmpeg.org > >> Cc: Jack Lau > > >> Subject: [FFmpeg-devel] [PATCH] avformat/hls: fix typo There is an > >> extra space in the original comment > >> > >> --- > >> libavformat/hls.c | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >> > >> diff --git a/libavformat/hls.c b/libavformat/hls.c > >> index 3bdc1bc848..c2130bb883 100644 > >> --- a/libavformat/hls.c > >> +++ b/libavformat/hls.c > >> @@ -1993,7 +1993,7 @@ static int hls_read_header(AVFormatContext > *s) > >> return ret; > >> > >> /* XXX: Some HLS servers don't like being sent the range > header, > >> - in this case, need to setting http_seekable = 0 to > disable > >> + in this case, need to setting http_seekable = 0 to disable > >> the range header */ > >> av_dict_set_int(&c->avio_opts, "seekable", c->http_seekable, > 0); > >> > >> -- > >> 2.48.1 > > > > Hi Jack, > > > > you're not working on a proof of concept regarding the > vulnerability of the GA, right? > > (just kidding) > > > > As an idea, you might be able to give that patch a little bit more > meaning by also fixing the grammar. > > > > Best wishes > > sw > > _______________________________________________ > > Hi, > > Thanks for your reply. > > I'm trying to fix some issues with hls. I happened to see this typo. > Since I saw in the ffmpeg documentation that cosmetic changes should > be kept in separate patches, so i submitted it first. > > Thank you for your advice, I will try my best to submit more > important patches. > > Best wishes > Jack Hi, please do not top-post (ask AI if you don't know what it is). It's a rule here, probably because in those plaintext messages without formatting it's hard to follow when some are replying at the top and some at the bottom. Cosmetic changes should be in a separate commit but they can be in the same patchset. Personally, I don't think it's unimportant to fix whitespace, spelling and formatting issues. There's value in everything that improves the code, but you also need to consider efficiency and think about the time that gets bound for others dealing with a single-char non-functional change. If I would make such commit(s), then I would go through a large number of code files (like all from a lib) looking for similar issues and include all of them in my patch, so that a reviewer/maintainer sees that I have really spent effort on it, and they feel that the given value is worth their time. Best 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".