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 8544845AC0 for ; Wed, 13 Sep 2023 00:21:47 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 6A4FB68C622; Wed, 13 Sep 2023 03:21:45 +0300 (EEST) Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-vi1eur04olkn2100.outbound.protection.outlook.com [40.92.75.100]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id D2EDA68C96A for ; Wed, 13 Sep 2023 03:21:38 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NJ9dnW8nKIZdsh8KSxVe+TtAGzQr0k7NheRLBv6tko1R5lcyWEFTO0HjhtI0pjuWaervtgu3wC0XDDwHI0V9MccaiN8X0uD1rsKIaecg18QsAAxAX26PEvIhuivNGaCI9azOd6qSssT1sCInMK03OC4lBF4nGV0/kdBl2OOy/PPbQOK7Ic5UgpAGV0duxbww7kXDT+5mGVcrE3fR6InKnQ2DlsbYiNsP4bRJLCn4g4nm+51EL2+4+istF6Wl5EyBD4YgC3A32LeF8jSGJeZh5WT9PjZsiw9S8XqdNiKcPKP1PGzNkli3V8IWrHzY44d6zpKoZlIUl7Rz+SIjbkLe2Q== 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=tH4Wih+GbIMPDCxt89EgryQSJO9nknk8Y+5hghpaesc=; b=CS/rC/rRG9VggkQZaGyrPOfaP82NICbe0aC8pWMbXeZKygSg3ziqDlszlUpt/dVpxiPvEP3S4+P/RPBawv5vURjr0bvQC5Jb7WR80V8TiEQgHFWEcC5+PjzEfRKpsUpC3ZZz0CIua9qsNhL52RwnzM+QBSAyd2l4yuEyv8nhC2Zs88XOH0WY1xFA6XkX0hpEmgs+WvzIo/4CZxpzrK4z1tboSiBXOCt/L0lMccLPxUes3NJQJeZyKf0Mf2a0F+yJ8A2lSa0PCcKM3EsShoq2N3XovyfvDkgHO4clxgd5xLBhD+4uc5sYnSqn0OrmkmXjUeBj8i3xoWwFzZf13JsmBQ== 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=tH4Wih+GbIMPDCxt89EgryQSJO9nknk8Y+5hghpaesc=; b=uwJfDUTh/sqenVZ1txr7P0xxdDG/2ClnLLC0GKoFkjAyj2prtrlouihupO9XYc3EOwo6NF2W41Log+Cm0kvu4HJCM560tp+dHVPEMVqqZYJJJbVZsPdCqTHaAQ0E1q9114hz5/ZT3M16XEb+CVQee8UrhaU7CtpNRmgskjzsZ5Lo1K39NuqFeNXnODYhR6/YQJOQPzUQOOL8CPKgu4qb6/St6VZkImL1OUpZRz3Aa1YRFqZ1ICmmBHPH2l0VuVHReIlIMio7Xq90RTR18zDv8iIOxQgDYN/yhFK3pBj6/OGburN4KsrpaKHah8atL6hmwDM9Zy7jjSd8xV+K6uXT1A== Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) by AS8P250MB0395.EURP250.PROD.OUTLOOK.COM (2603:10a6:20b:377::14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6792.19; Wed, 13 Sep 2023 00:21:37 +0000 Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::3fff:eb7b:b8e2:4dba]) by GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::3fff:eb7b:b8e2:4dba%4]) with mapi id 15.20.6768.036; Wed, 13 Sep 2023 00:21:37 +0000 Message-ID: Date: Wed, 13 Sep 2023 02:22:54 +0200 To: ffmpeg-devel@ffmpeg.org References: Content-Language: en-US From: Andreas Rheinhardt In-Reply-To: X-TMN: [7YJyV4vpfTbSOn0OmtOESXB34uVMdbMn] X-ClientProxiedBy: ZR0P278CA0116.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:20::13) To GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) X-Microsoft-Original-Message-ID: MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: GV1P250MB0737:EE_|AS8P250MB0395:EE_ X-MS-Office365-Filtering-Correlation-Id: bd39d108-2f86-403e-4624-08dbb3ef64ae X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: jXnPu9r6mIUvc9TRGhobyAc3+CAEuXHyczdwT6zEfliAKYirgg0Tv+duFFNiJdccRB0CbZFjfysMr5xwoRqULY5yVjNxX2RjkdEs8dsqfw8m52XKRRzfRH1LD+GdjY+7cCNyqOL4J16+K7lLkzO5TmYE75auLXE4YaZTdiosfhqWEpKRy3HwEjcwRCflEiG4L4MqUrSyuZIZDwcYsVLNLQ0Sse4PtTGz9R0oPxAqhtXBie9e/6DgUnYVWirs+zUwXwkL4JUuCwFlg8Kc5XXWqNMGweE/6QBMCnv4sBH+rRF6UWbd4yuNqjIUaLjEGKgBED17tMCULoRxu5NoTqB1QxHiTRztGgpJuxZvmUuC76BAgyet2KEErtICKJ5DGhSUAd3mkqqsVs4sL5ZksK+7nlVlXHWsLLdgI3lfcr/iDWfj+BDa3Jo7RMeO1s4Rp0QFUHqM1vRW3YQKmV0jYA8dPwyKWOnSO2MCp2did70tUC8mRJqmBkRG9nNoXPSfY+5jjmq0c1KKOJja7mz6KSg/7R4IreTudKFvUi+OpXqiriohtUNOaPG17b8s0THnWTZx X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?NE1ya3VBWkdvZVM3OUxFWVFwZUphU1hack5DdTZON1UyUVVwR3FtQWpCQlZa?= =?utf-8?B?V05RN1VDSFliSmhxdmEvdkpkaDVVaTQrM1RSZXI5czdHTGRkaFZpZ0I5MFo2?= =?utf-8?B?RjRHblhRanlPMXJiZ0NJV05aaDRZbzBCVWpEUDI0YmU4aEJ6ai9jeGRBTlo3?= =?utf-8?B?NXpMaStqd2hVbTZienk2ZEV0UzEvZlRHSytITWhYU3k1WjZWM1Q2V3gxdVF5?= =?utf-8?B?NTJDeUpVSFZSdmlzbXBPU2Y0RndLVVJxN0swRUJnZ0owVEpoaEt2SXVYclQw?= =?utf-8?B?WnQ0dVZocmhtb2VaRWhTYm1IRmtRaHZqNWROOVI5OGt1U0p5ajVqNEk0d0c1?= =?utf-8?B?anZlTy9SdTgzR0R4Nk9wVWdZeThIK0RJT2RoS3hQZlJzek1ab082R0E4VzVt?= =?utf-8?B?ZnIycUdqTjBBMXZsWlJicWZsdXBmc0paUzIvaVN2WDB0VFd3N29ma2xPcVhF?= =?utf-8?B?ck53MEc1NVRvWE5KUXRtc2NnT0tDYjBMWURWKzMwN0c2ajJESVRManFVNjE5?= =?utf-8?B?T3F0T0VhOUduRGFUQzRnSG5OV2FNOUs1UXh1R1NRRFdKWkFNZTEwb2Q3TU9m?= =?utf-8?B?RWw2UTM3Y2lGVUFYQnY2Yld6cWNXZUJOeG13SnFuamplb0hiWU9RVWVUZDlT?= =?utf-8?B?ejlTOEFRRk8wMVQxRHhJYTJROWgxNVVYSnhWbHFqNlNmSVRUOFVPbjZ2T2x0?= =?utf-8?B?Qk5GT3RMdityRmp6enlsNWljZ1N6V0tQVm1zTS9kOFpiRmpyc0J6WDcxeGY5?= =?utf-8?B?WnhvNDlENTZ1cThySUhHVXk3MlVMSGVCaTViWjVXeWErRFRtTVAyeG11elhI?= =?utf-8?B?UmVCVHMyNTdScU5JWWdla1UrNTNjRVhyNHBvbkVCL3ZtWjl3cGN3b3ZxdHl6?= =?utf-8?B?S3hTKzNKc2xGeGFwWnhGNW04M3lFT3NYSmUrdGpIQ3BSeFFuOFEzT3JXdUw0?= =?utf-8?B?YTFMUnowQVgyVDRPaS9ta0tnaTVveUUxZHpUbHFnbG01amYvc05hZkFpOGc1?= =?utf-8?B?cnVZSHVVSzBEbGNBbWkzdGphbXZsUGF1bXIvb25wKzNTRlZJek5aUkVwSXd4?= =?utf-8?B?M1B0VHhiSjd2N041c2tNczZLa2JXYWdheGpBS3c5RnlVN3d1ZjE1TnZXK2lC?= =?utf-8?B?Q3VJZHNseFV5cU5JVXUzYnVkRVEyYy9idUJsUmFwMzdwY1pkZGlIcjBQYlU3?= =?utf-8?B?eko0RlBFOVByTHF3NzVVZ1dXUWg0Rm10YXVpUlhVYS9zR08wV0wvUUxoaERY?= =?utf-8?B?dGkvbllrUzl0THh5VWFiMFZKYTA5c0ZnaU1LUExZVDVDcVRMVW5mV2tBU0c3?= =?utf-8?B?QzFpemtHYW4xamhYQkNoNy91a2d6UWR2QXE4NFczSFBxWjVrSXM5YWYydXdv?= =?utf-8?B?b0ozeUpIOFE1L1FmS3Y5VFVnS0FjTldqWlFmMHY1elRQbHpWSGM5ZHBwTlVT?= =?utf-8?B?NTZNTXZKcmFSalcvTndqTlpWVE5tTGhNaUYxdERGYXJTWUlZY1Ewb0FHdHcr?= =?utf-8?B?azlKT0dRTWlvSHlaMlNXcVdTYnM4V3FWanNPbFIzQ29yL2FNQlBaSHRHemoz?= =?utf-8?B?bkdLYWw0VUI5aVlvSy96Z1RoREVrMy9KQWNiNUQvZVM3V09SZWpvNW9lRVlu?= =?utf-8?B?dWhUc0F1TGxlYVdoakJoK3M5YWxBU1NUM09VNHkyUGhDRHh4Z21RQ3NzMWtp?= =?utf-8?Q?I0BiC6Jp7vNYFT0kDvFa?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: bd39d108-2f86-403e-4624-08dbb3ef64ae X-MS-Exchange-CrossTenant-AuthSource: GV1P250MB0737.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Sep 2023 00:21:37.3594 (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: AS8P250MB0395 Subject: Re: [FFmpeg-devel] [PATCH 1/3] avcodec/av1dec: Pass AVCodecContext* as logctx in get_sw_pixel_format() 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: Andreas Rheinhardt: > It indicates to the reader that said function does not modify > any state. > > Signed-off-by: Andreas Rheinhardt > --- > libavcodec/av1dec.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/libavcodec/av1dec.c b/libavcodec/av1dec.c > index 8f9c2dfefb..8f6c4f732e 100644 > --- a/libavcodec/av1dec.c > +++ b/libavcodec/av1dec.c > @@ -440,7 +440,7 @@ static int get_tiles_info(AVCodecContext *avctx, const AV1RawTileGroup *tile_gro > > } > > -static enum AVPixelFormat get_sw_pixel_format(AVCodecContext *avctx, > +static enum AVPixelFormat get_sw_pixel_format(void *logctx, > const AV1RawSequenceHeader *seq) > { > uint8_t bit_depth; > @@ -451,7 +451,7 @@ static enum AVPixelFormat get_sw_pixel_format(AVCodecContext *avctx, > else if (seq->seq_profile <= 2) > bit_depth = seq->color_config.high_bitdepth ? 10 : 8; > else { > - av_log(avctx, AV_LOG_ERROR, > + av_log(logctx, AV_LOG_ERROR, > "Unknown AV1 profile %d.\n", seq->seq_profile); > return -1; > } > @@ -467,7 +467,7 @@ static enum AVPixelFormat get_sw_pixel_format(AVCodecContext *avctx, > else if (bit_depth == 12) > pix_fmt = AV_PIX_FMT_YUV444P12; > else > - av_log(avctx, AV_LOG_WARNING, "Unknown AV1 pixel format.\n"); > + av_log(logctx, AV_LOG_WARNING, "Unknown AV1 pixel format.\n"); > } else if (seq->color_config.subsampling_x == 1 && > seq->color_config.subsampling_y == 0) { > if (bit_depth == 8) > @@ -477,7 +477,7 @@ static enum AVPixelFormat get_sw_pixel_format(AVCodecContext *avctx, > else if (bit_depth == 12) > pix_fmt = AV_PIX_FMT_YUV422P12; > else > - av_log(avctx, AV_LOG_WARNING, "Unknown AV1 pixel format.\n"); > + av_log(logctx, AV_LOG_WARNING, "Unknown AV1 pixel format.\n"); > } else if (seq->color_config.subsampling_x == 1 && > seq->color_config.subsampling_y == 1) { > if (bit_depth == 8) > @@ -487,7 +487,7 @@ static enum AVPixelFormat get_sw_pixel_format(AVCodecContext *avctx, > else if (bit_depth == 12) > pix_fmt = AV_PIX_FMT_YUV420P12; > else > - av_log(avctx, AV_LOG_WARNING, "Unknown AV1 pixel format.\n"); > + av_log(logctx, AV_LOG_WARNING, "Unknown AV1 pixel format.\n"); > } > } else { > if (bit_depth == 8) > @@ -497,7 +497,7 @@ static enum AVPixelFormat get_sw_pixel_format(AVCodecContext *avctx, > else if (bit_depth == 12) > pix_fmt = AV_PIX_FMT_GRAY12; > else > - av_log(avctx, AV_LOG_WARNING, "Unknown AV1 pixel format.\n"); > + av_log(logctx, AV_LOG_WARNING, "Unknown AV1 pixel format.\n"); > } > > return pix_fmt; Will apply this patchset tonight unless there are objections. - 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".