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 91965431C5 for ; Tue, 24 May 2022 18:51:06 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 5FE5C68B4EE; Tue, 24 May 2022 21:51:03 +0300 (EEST) Received: from EUR01-VE1-obe.outbound.protection.outlook.com (mail-oln040092066104.outbound.protection.outlook.com [40.92.66.104]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id D546668B4D4 for ; Tue, 24 May 2022 21:50:56 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Fh98N9a0agJvOQBkmF6ISXrr4T4slecioivQ3EG0BLzTrNngcev1BL4zjv0Yxi5fq4+Rl3LMiKDkgOGyaeLp3dc+YGOstF2nWq/rlVKSuq4AAXiwB/+mSOXAJSecOkMuoezhL3Za8t+eY9BOKzI3PAvkYoplRwd97Peiv7oXtVAPvcW54UjhmpiHNgctqJ74nle0aONPXaLSBOU8cBxWTFLqAi15csXrOYqZZEbN+JXmAJNcDuE0xJsmAPIaR3b7BV0yxVuJHxiynTUZJ+J8IfPmJon9IdUZOKnVaqxX6nhqnQJqfZ1Is9J/xLBoQTYr5eLdhNpJeEGZ0C6iEas2qg== 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=jXZKG5Dre0oXafTLbh/c+7Z06eiXgQjdJyeik8p2svQ=; b=LPTGRTwahtyGTIKIdMIX15OYoOpE49Ay2zPxjYfUNRpLSs8dERScTkbq237tP/e765A1SRH6xPozM83rGkR7xxVxR2tcYYXvaW2tTFJgatB3Kmh+qRip2kwYtAlZfZJ5rOY+aSjiOlH+6cVT+AQvye8mFv1b4RuAO9de8unGITvDNe/UQ0pRNvy+wkFwPstuqfFE3oygdBn+5z+3mJzkcOyZjPWpskg3TmtQch73UrUDY1T+6pBOUxkz1Gc5vnc3yvVGrdxuWljssDEmOvF/BhEryt6A+5whrxpghPX78MYNT43fYPw+Vb9bB0GImxbnFq/JWPT3pRd7D6Ro4kzILg== 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=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=jXZKG5Dre0oXafTLbh/c+7Z06eiXgQjdJyeik8p2svQ=; b=l9/s9bEL/FXX/mlcZDIz5CQjdEoC96wwrptTsh39uzf/jPH9GjsGS1igxXgdwKArSBrxRevsHRCoQ9rx8+lVo1pwkSXnJg6gKHe9P8Xs9NNQ5GbK2ldyjwR213DG1GUyaBEJQMEdLt3mCvi5pYk2fSD/YZ9epyoA68B5HVxDnEjVfxtwOz0kx2BkvugrL8OLODJAba3meVqGSVJzAgKy2aLCWUKmV+pcb8se7MbLWEhrS9UfkdSDinvQly4S+lMTUVVxZ8nUx/Mt6JSdxszdMLZuJAm2QFChYI1KeOqTtBXNWwUHLbyyADdxpAIVzsgYXzRMVA1eQyCKUi4qYA4ZUw== Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) by AS4PR01MB9494.eurprd01.prod.exchangelabs.com (2603:10a6:20b:4c9::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5273.23; Tue, 24 May 2022 18:50:54 +0000 Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::60b9:9f29:40cc:f01c]) by DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::60b9:9f29:40cc:f01c%10]) with mapi id 15.20.5273.023; Tue, 24 May 2022 18:50:54 +0000 Message-ID: Date: Tue, 24 May 2022 20:50:51 +0200 Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: <20220524000615.5264-1-michael@niedermayer.cc> From: Andreas Rheinhardt In-Reply-To: <20220524000615.5264-1-michael@niedermayer.cc> X-TMN: [wa4Zfgb3mb5AF8Hu4p9PsS0rKfCTY90T] X-ClientProxiedBy: ZR0P278CA0101.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:23::16) To DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) X-Microsoft-Original-Message-ID: <9a0504e7-02c1-af07-d7af-28962596a0f0@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 1038b78a-8213-4c68-a999-08da3db65486 X-MS-TrafficTypeDiagnostic: AS4PR01MB9494:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: nrq1VLYv/U0Nhssr7cfT28fgBzToSTcS4u7cj4NL1jej9qDAnOY90eu+Zdt1I0Ttkug+92bqn+frNnpFT06s+Jc1mwj0sSwRvDYzhMbFF4xPXZKj4HRGgQfYhXSs1g0gXeOC9ENi/Yj81JGcsZzE6gQYP0t7FLs5Q8Q0e/bqKHYyUI8YBd7TLcY1cgApBKeglUPtrstYCLgrBbiYSjQL7tQJmSrMhdNEcv17EAZ2Hy0qsupTrQ72s7G2ff7Pk2d6uDTS0OGdLrZCZDJV/6Ydbc7oABT+mfe+FSvuFyERf5aAvgMXism3ioWXYnblVeVKYpkIGLVXI7l36UL60f0Yg9sTatTnMxNGxZqFZfHfYpQvYL6kClyikuagt/hC2fPzSYKhDaEEL4dkz/wWhazRbwRhFwo7teb02reNXVWMytC6Fry2wu/gaZw7aDqH352Sr4YJRZ/OTf5RmKdQN9jg90cJ7ZVZe23GrOB+NlHjXDbguPquH1hdS7tyz1UAPtYiTJmdh6s3WYVPyELxGC23jnwUpVnHoodTF1pi9Z9272+dxlzdPA3T78YsSq5kv7S/CHERRUwzRon6l11/9hgI0w== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ZnArRXVqeEgvUUVqeFM3cS9BemtoRjcrRjN6ejFWZjZsaThmSGc1eWpuQnlR?= =?utf-8?B?Q1loWUFveU5KU3NQcEJ2VUxlSGEvU2ZBc3R2a2duQklBMldxM2RnWWR1VmhS?= =?utf-8?B?OXNJWW0zaUFzRzRubWR1MktWcHI4WUpmaFhpaTlxVi9zYjM1MnAzL1YvYURl?= =?utf-8?B?cGthN1ErWDZXeGUzbnY3cmNDdkZ5S1FsL2Yyc0R4bGI5WUgxZUNEY0pUUmtx?= =?utf-8?B?ZkJDd2luWCtaQjlpUUFsU1I2QmtZTjFkYmZya2xjSUlZTkFFSVFCMkZBdk01?= =?utf-8?B?a3VUUjd5WURGVWJLS05EVjRSS05CcEF3a1dpWWpPVi8rQmhwdTBubkFabkc5?= =?utf-8?B?dktVZmxmRXJjK0xKa09RRzhwU20yd3AwZG9YT1gwNjZNaURvOXZFdDFtSFVk?= =?utf-8?B?Tmt1K1JSK1c5VGdrbmgvRWkxUWRpcFhqSG9yZC9EZXZhOHRBRUswNmZZV0Rn?= =?utf-8?B?ejFYbEIya0Z1RzV5M2s1UnlHVzJRWGdTZE5QeEFSckxWWkM2VzVCRU9OQkN1?= =?utf-8?B?WVc3d01pWUNuZnJFeGhlcm15a2JERUJNWlBxOVMyWXBGWGZNSXp2ZVkwV1pk?= =?utf-8?B?ckQ5Qmp2aVFjaXY5bzhsVUJ3WGRaRC92SnVvRmxFcWFSTUZIQ1RwZGJRdk1y?= =?utf-8?B?QnhaM3ZyR05OYzNKOEVyOWM5UWJHa3ZlZFNsUEpNMEZlTnA2WUQxTU1jUVpW?= =?utf-8?B?UnRmK0VtNHhkcE9VSE1hUGM1Q0JXZGp5NHJSeFhEdmFRTnpQMk1qZnBRSnJ1?= =?utf-8?B?aENGSTd1TThjRi93VVM1NWgweWRFOGE2QloxZUpSOFNiNUdPUFVTMXRhcFd2?= =?utf-8?B?NlVVa0hXYUhFRVN6STJvaGo4OG1rd2VaUURtZER4WWNXY3k4c3h3RWJ5WWtV?= =?utf-8?B?ZlJ0QytKSCtJY3FPTFp1RnVvTHo3OGwzK1NvU2V0bC9SUWdrd3VkNjBnWWcv?= =?utf-8?B?TGdJKzJIU3NwWFRhc0pxeHdWckFSVFhmQVhRQkJpVC9GNUhVa0JuZjIxeXZ5?= =?utf-8?B?ejZ1b0FLam1EWC8yNmlSR3FHL05zcGt4RXFCalJvdzZRbjh0eS90WVZsVm5m?= =?utf-8?B?WGhVNFZNWjM0czR3ZjdsKzdOQzN4a2JEU0VlM1FoTVkxUHdnZnJJVE1ReG1r?= =?utf-8?B?RHF5MWtkTGdmUnEwUXpsZzY0cHF0TVp6UDVocWRJZHRvV3RXSWZMakpIRlB3?= =?utf-8?B?cmlUNXczMldMY3FWcGVsM3dZL0VtRlJmT0FPY0daWmQ1SDlNcnJHTzRCeGxK?= =?utf-8?B?eFUxV3JjQlRzMUZjWVVQQTZwMnRPYVZ5WW5zQ2VoeloxMVdUWkxWV2FXaWMr?= =?utf-8?B?MkZSeUY1YUV2TFUvN2Vnb3J4OUQwSEpDeDhiNkRmdGg1ZU9WOWNNcm81MUpC?= =?utf-8?B?L3dhZ1JPTFlzaitSdVZPYTVzeXlic0NuMGU3aldYK3BGakd0ZzNuUWJSTGZT?= =?utf-8?B?T0RtSzI4MDVqbkgvM2l0Z25qbWUxcU5OQk5EeURUV3ZueERaZ0IzMjRxNjBO?= =?utf-8?B?YmM1OHRMa3JvekE2ZEs2cnQ0NmRHamNKZGVmZ2pMZ21sYTNqYWIrOUJMQ1VG?= =?utf-8?B?ck1WcWVUTW5QZXUvVXZVKzFpak56SEhHWlVCYUhYdTMvSGhGalk5aUNRMHBP?= =?utf-8?B?MDBYanlaNjR0eDdRMFJhbG5GaVZ2M01RL3pBSlZYU3ozN0laMlZJdlNhVjc4?= =?utf-8?B?ZnBKWUQ4TUtUWi84NUwvSkZFT1dwT25LWjFxNlg1Z2JCVjNvR1hLTDhpekYr?= =?utf-8?B?d25ERTQrWGVIT21LS3JXYk9rcFlqZ0x4WVhEK3hucWlOK2pyM0ZGZE9IVFZq?= =?utf-8?B?cnc1VGtCWSt4R055MGYzT2p1WkViUjM1UHRndnZ6N0o4S2ZnUG9rRGJyZnNw?= =?utf-8?B?V0phMkhWeFZkTDhOQ0p1eVEzTDJ6eFk2a1IvUW5rMFVNKzZ6RkVFNkJ2V2xS?= =?utf-8?B?QjVKcXNVRU9sMDAybzRYMTdRUkRXSThqdVFlY2VpMTFEZ01ZMG5vWjFvOGxP?= =?utf-8?B?ZmVXQXpuMEl3PT0=?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 1038b78a-8213-4c68-a999-08da3db65486 X-MS-Exchange-CrossTenant-AuthSource: DB6PR0101MB2214.eurprd01.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 May 2022 18:50:54.2956 (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: AS4PR01MB9494 Subject: Re: [FFmpeg-devel] [PATCH 1/2] avcodec/vc1_loopfilter: Factor duplicate code in vc1_b_h_intfi_loop_filter() 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: Michael Niedermayer: > Fixes: CID1435168 > > Signed-off-by: Michael Niedermayer > --- > libavcodec/vc1_loopfilter.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/libavcodec/vc1_loopfilter.c b/libavcodec/vc1_loopfilter.c > index 0f990cccef..ee694ede28 100644 > --- a/libavcodec/vc1_loopfilter.c > +++ b/libavcodec/vc1_loopfilter.c > @@ -1125,10 +1125,7 @@ static av_always_inline void vc1_b_h_intfi_loop_filter(VC1Context *v, uint8_t *d > dst = dest + (block_num & 2) * 4 * s->linesize + (block_num & 1) * 8; > > if (!(flags & RIGHT_EDGE) || !(block_num & 5)) { > - if (block_num > 3) > - v->vc1dsp.vc1_h_loop_filter8(dst + 8, linesize, pq); > - else > - v->vc1dsp.vc1_h_loop_filter8(dst + 8, linesize, pq); > + v->vc1dsp.vc1_h_loop_filter8(dst + 8, linesize, pq); > } > > tt = ttblk[0] >> (block_num * 4) & 0xf; Are you certain that the current code is actually correct or whether something else was intended? - Andreas _______________________________________________ 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".