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 0190A4934D for ; Sat, 9 Mar 2024 19:38:00 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 09D6868CCF8; Sat, 9 Mar 2024 21:37:57 +0200 (EET) Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05olkn2025.outbound.protection.outlook.com [40.92.90.25]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 2E15B68CB56 for ; Sat, 9 Mar 2024 21:37:51 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=YeCCva+pCWL5PzreB4B32WBBVd6Z6kCevKjhLoPmP6fIqbFoZbUIrpEYz5DJ5/3YR0U88qiU6FsSwwOPQyER5ytFnfbC/Ocdd3wOJDdYidUP2CZRcTDwqktggWQmyRm94grtqNXoaSskyCU+7So8210vRW/CD+mOUFhO38AkZggQQ+NFkgU7SaZJVwFkhkqoyjMHKv30DSPNg8zmnd6nZ629lyNkAfQO7eD2JCKhXcnIYNIPXo9koFt3qbuVEMkNQ+YxRoqVrtltAvYhaTh9xuf/2ks+M1ORrfMo/uTdods7+R17qiJQAVnIgVGwAElEoFWZsU6aGlLLlStcbh/9lg== 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=wxfhzN5DwJV5M8a7IlMelLuIQ8Q/rGBIrfpQom2Bvlc=; b=egx6xLSmqKaVWh9Ulgj3ngpIt0V4MrUqIGhh3rY4YUkbXTw3w/QvuyWckgk+2KZRVc4K53mlYxtUVuQ7g5dQRlz7lpznypGlO/oN0zmG97bjEWB/yeeWFoljtrFCZr/3tPYWzej/22cq0g8DkDBNZuvM575dS6+QovyrUuK+LZCyHC+bh+4jt8zqvtaLLgYF2/U8OAZRJ3exnFXK7YZk95pGKO101rYmN9jx/Lp8Z5voP9fQK7qg4QDt6BJQHntpBFHlk3BSSLbmq9lH4IdebP3uyw6vwBYcx7MNlFt72R2t9J/+O7+TIt7YbfYOpWd+XtTLKpv39bZb/mPl7Mo8rw== 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=wxfhzN5DwJV5M8a7IlMelLuIQ8Q/rGBIrfpQom2Bvlc=; b=jasLh0wp3MepLo/2YfUeN4EQwZFM0kTwq3g5DLGrxE5C5vDkqc3Y2g0nywXviommU8+S/DRieSsz3wmQQ4tqLc+ynmOmwgvfBn7pegZP8R1+nDtH24e9ifsYfvJhUASB2YbGd7NAfVnWMPD7Yii+Tnjtbc2053EhCrGn9jvzzC5t4IAePgm37aChnKA2iFjwhrvTPA/uTz/l7H7rODhXVyecmVSVE3vgLMRsopiUVOaxJOzRvMopn3LSFT385OFQOe0DgudZLRbqJ1bcvrcEoKVA8DlkBOhOMUPC3aIPebn+OxUbfVtvuPz/xuZFu5ACvS0Zgasb+kROXQ/4grRZKQ== Received: from DU0P250MB0747.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:3cb::16) by PRAP250MB0614.EURP250.PROD.OUTLOOK.COM (2603:10a6:102:294::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7362.33; Sat, 9 Mar 2024 19:37:49 +0000 Received: from DU0P250MB0747.EURP250.PROD.OUTLOOK.COM ([fe80::8230:ffad:27e5:20bf]) by DU0P250MB0747.EURP250.PROD.OUTLOOK.COM ([fe80::8230:ffad:27e5:20bf%3]) with mapi id 15.20.7362.031; Sat, 9 Mar 2024 19:37:49 +0000 Message-ID: Date: Sat, 9 Mar 2024 20:37:49 +0100 User-Agent: Mozilla Thunderbird Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: From: Andreas Rheinhardt In-Reply-To: X-TMN: [LgYqdPYAkBV6GllFwiVyEvXj5MbEA/GG7lJU+rqcieE=] X-ClientProxiedBy: ZR2P278CA0066.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:52::20) To DU0P250MB0747.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:3cb::16) X-Microsoft-Original-Message-ID: <81e1a79e-3752-4d6c-b2ee-335719b9be4a@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DU0P250MB0747:EE_|PRAP250MB0614:EE_ X-MS-Office365-Filtering-Correlation-Id: b97d9a16-84a9-4b0d-c484-08dc40706783 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: PwOo36ZiKv5b1vTmdUzsEMjdrbuO0ZJMjql9AcxaI8CSDWaakaHYiAAr/cwtkN7LbfKoaV0xBlwHEI4MuG8SxptWSNQr0cITBNbTDLkaTiZGx26tfYnatl9NMjgSSTko2fpb77oGYxP4agQMCnuyIljknGjIFFtRO/gOjBWU0unK/aXIK9MqfSRmCVHa3j/i2fp8kdDWG5XE1+cviALwhB14UnxpoYIoWAfkD+J0h7XA4M9Ev3LsU4FRTnUBP9zteKN8kCojphu+bu6PngZJVc3BjG8CS1Wj7heK6w3Y39yCupP8WWjSjSGVqNIlsZ7COd4QhvuVlr+kxSSB8yYQaFSoiLGHKY+GmMqLScj45p78m6GNCDjMqfvZrTZ5oC5cLBNAouQiTIkKkzh10mZjBDiUl66O60EAcGz4MO3C1BX6OICExv34oosNQbzGJhyTwMs+3/H/knZ0Uo05bSZG6yhp4aCd/FKxkM/zk42qlmsWWXZfixkI8lZDMrDIxggoBXQ6d44y3kFFhW+dpgyKv7h1tcCVHIUg2zICH04MtF3jOQG58981Pv3SGrx1V8xg X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?R1hBVmZrL2xnT08yZVpJbGsvbFpqWGlXS2J6WjQyWXlqZVZLUE5VOW1kakpO?= =?utf-8?B?Uitubk8rc2pBeksxZWt2a0gwT0p6cUpWT21mOThHSHFEaHVMREt2UzhYbjR5?= =?utf-8?B?YTZQNHcyRzgvOFZWakF0VUZBa3hteTF3Rm1PbngwZWRDeFhOc3c4Z011RFVj?= =?utf-8?B?SVF4d2FIVGNBZ0QwT2wvczhYK1dnTVd3Sldmd3dXcURpUUJJWnlKRFpyT2VE?= =?utf-8?B?aTU1cVlISTdwZDJQWHhyd0NScFdNTm9JZHJZSVJqMTlFVjQrZDAvZlBUTWNQ?= =?utf-8?B?UElqRW53Rmx1YnU1aHVhQUtSRXNZZ2RpNWx1QVY2N0pJOWhIS2JCNldXK2FP?= =?utf-8?B?cFRQUTlWQzFPSDNjSmlUckhzZHhpdTNwVTFwMlgrdjFCeGoycjJWbzZaaXJ3?= =?utf-8?B?RlV2ckhSR1o0a1drYkpndTJFTUtGUm1RR3RrRXlGNHh5Uzd1L1VxMHZjRTdL?= =?utf-8?B?QUc4NGlrd3pCbndmdUh5SCtuWm84WktzaUdtRWxBUURrU054Tm5ObnZNZ3ps?= =?utf-8?B?MnhITklha1dlazhlbCs2eHdmNHFqQWJlYTl3L0crUUhxK1RpSkE0byt1K05R?= =?utf-8?B?WFR5V2FCd2hBZENPK0pqYWs5V0tKRC9Rdm56ZzdqTGViMFY3M3ZnU0pnRnY5?= =?utf-8?B?Yi9iY0VWZW1vMXNGakxlMlJzaWlZZU9zbzJ6T2lOMWpGL2tlTUF4a1hpRnIw?= =?utf-8?B?MUY4Z1lROEs3aTlJaXdkSGFVbUtDZVhPaDdJaDdZSjNlSUJKQWxwWjZ3ekx0?= =?utf-8?B?VG1OZDdTcFdsT241ZlZDZXhyMllZRWhDMnFodkIwdzQ3UmoyNXpwb3FHcXBN?= =?utf-8?B?SjVYZU5RTHFXR0xqVGp2L0ZrWVdQZzBMcVdVZlNmQWxGVXVMdDhva0Y1cE1K?= =?utf-8?B?b2VJejhpL2pwRU1HSzRMeW44cVBySy9ZODRBdXl6SHpHekM1TzVNY1dpOWhE?= =?utf-8?B?OWM1VFlZQ0x2Qnk3b3poRXVNMFhWT0tQMVh0U2laeEh6RFVNUkxpRmJJc3Vo?= =?utf-8?B?czhEY1dkSEtaSjFCcmErL3M5bGRrMG1WcEx5VXpaNjVyK2dYdmFoRk5hSFh2?= =?utf-8?B?enJ3dFd1elgwUEpocXFsTVFrUkZBS2JIU09nTytvZVRXNU5QNzAxZEM4Y0dr?= =?utf-8?B?WlhLbnpmd1p5RGpyZE9OS1ZHYk9CQjRKOW9KV2JEZGdvemFaV2c2d25UUDgx?= =?utf-8?B?bllsMk90dWFTSTAyaGZGM2pYVng1dlJHeDl6MDAvVUFwdUQyY2tuZ3JBNkZs?= =?utf-8?B?N21xaHFsQi91ZmZPN29YaENzcXhHakxBcjVPWStKdHZ1djc3eVJUZkYxVkE0?= =?utf-8?B?MkJ6NlFkbUJzb1hGeEtsMDRDVE16Q21idVVDM1JFTmdhdlJXbzA0VnRHU1or?= =?utf-8?B?aitreEFrd0d1RTd4T1JCa3BqUUJheElQcFVpL3ZucWFCSkJXOEp1dXFmZ0FP?= =?utf-8?B?c3NUOVdFT1g5TVhrSVZFSjVZdkE4dVpHRHlHMjFlL2N1amllZkMvWnB2RnJs?= =?utf-8?B?dkNnZnIzZXAzeUwweElmWCtEd1ZSRlpoR2xuYzEvQnM1am5JUldwTGJrMk5n?= =?utf-8?B?MXlzNUpnUFpXb0cwdmRNUnVEblEydG4wUHZETjJ1MlE4cGVFMllRMDNlSjJC?= =?utf-8?B?UmRBM29OVWxQQ1ptK1JhcVY0aE1sakoxTFEwWHpaZ1I3Q0tBM3ZyS0MyZFdN?= =?utf-8?B?b1hyMmhGVWdxVHJEZDlrTEhtM1JQZElUS25yQkVnS0t1V1QxUjBQYUZRPT0=?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: b97d9a16-84a9-4b0d-c484-08dc40706783 X-MS-Exchange-CrossTenant-AuthSource: DU0P250MB0747.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Mar 2024 19:37:49.8492 (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: PRAP250MB0614 Subject: Re: [FFmpeg-devel] [PATCH] avcodec/librav1e: Don't unnecessarily create new references 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: > Signed-off-by: Andreas Rheinhardt > --- > Absolutely untested (not even compiled). > > libavcodec/librav1e.c | 8 ++------ > 1 file changed, 2 insertions(+), 6 deletions(-) > > diff --git a/libavcodec/librav1e.c b/libavcodec/librav1e.c > index dbd728a408..2a6d8bfbed 100644 > --- a/libavcodec/librav1e.c > +++ b/libavcodec/librav1e.c > @@ -472,12 +472,8 @@ static int librav1e_receive_packet(AVCodecContext *avctx, AVPacket *pkt) > > if (avctx->flags & AV_CODEC_FLAG_COPY_OPAQUE) { > fd->frame_opaque = frame->opaque; > - ret = av_buffer_replace(&fd->frame_opaque_ref, frame->opaque_ref); > - if (ret < 0) { > - frame_data_free(fd); > - av_frame_unref(frame); > - return ret; > - } > + fd->frame_opaque_ref = frame->opaque_ref; > + frame->opaque_ref = NULL; > } > > rframe = rav1e_frame_new(ctx->ctx); Will apply tomorrow 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".