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 1E63840625 for ; Sat, 26 Feb 2022 07:48:37 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 31ED568AB7C; Sat, 26 Feb 2022 09:48:35 +0200 (EET) Received: from JPN01-OS0-obe.outbound.protection.outlook.com (mail-os0jpn01olkn2068.outbound.protection.outlook.com [40.92.98.68]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 2E39368A920 for ; Sat, 26 Feb 2022 09:48:29 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=f371rlzI2uz4c1PgW+ZJKe7bfGSrQLKsci0bmOy0nG/PpikjBbS4CTv2rahAKSzh2WJCJ9HeuLamsbG1WOKeR5BiMYIjWueWW7pKjScTkNsm2J1FEL7Pqqmf32tATQ7hgTrEl9LKBrFNpuT5lvo57RzsYO330+2W/gZS3iyGe7vYPNNDbHgwDuxRlSzrU7uSUPbP1oI8Nqnt8H60IxnZA9a5b2mJ/7Ziiv/a+fO39ldmWbXKTL+H22bPOmNqIFALKwwKbPVyQIJK77BwsptriQF6bV//pQpBuI73kT5/68b+i07d9u7UaWjRxl8hpOJHdPJYrs4zetI4CcF5cNhngA== 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=pTklNHX1Y5btx2H38nk8Acepkv4Ev+5DMQSneHEaDCU=; b=VwQ3rtYMOuNbWv2Yxmr7lmmukR9aZBbv50FsliadJT5uWEMMT/60trxRHE9nxisHJXqHawtWqI5w8CMG7Gt8CuOlmIIptbzvJgSvV9rmJn2OJsOCYgAKrKisXEf53yKiDCUnDFNULUyzA9Ayzg52I8k1JJvb5q05qoVAxwcyKRdhWnFvf1U9kJLNHLPCdudvmeTBOZF0IhOoJT2wjyjeBy+KlNBbYu7n/n8URscUeVCiOPNxrtCrIoq44Nm2dpJo+gLwzi9P42wYELgODe4MWCiuDzgxCzP2BQW+HFcEBFZBh2KkOSPseCiFcsuBjA5fIPy056yhbyanpii158AdOw== 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=pTklNHX1Y5btx2H38nk8Acepkv4Ev+5DMQSneHEaDCU=; b=ByWOwzVkz6nTYB481QVbYy4/NZvUxNvOmwIDs6fUP6Xo6B2xyezgEdmcnJU3E4grZqUsVEqZBFwk0gfYyzkleRMR6hiXdS+V3vZXaQRxXZxJJlisLP0WKIMqGIVGcNMTnsDfW52NKC+RJNRmt1paAwFuy2hEpi4rqmmdNzF4EZ9QQxN8uQJb0XSLdX+eMs78ZyL4EEkID/qIcA6AF9QOVLAhT98KhLyLuFQeMJc2JRTNcP/6M+QBWwbGup70M167E54YQKkYxekJoFOsNNSIsOU2t/670fpTzyW41eeOV0hpwYlcq94AmXGQYdDXaJa3/umPvTH64K8QkC6PIkFoZw== Received: from TYCP286MB1969.JPNP286.PROD.OUTLOOK.COM (2603:1096:400:159::9) by TYCP286MB1106.JPNP286.PROD.OUTLOOK.COM (2603:1096:400:b1::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5017.23; Sat, 26 Feb 2022 07:48:24 +0000 Received: from TYCP286MB1969.JPNP286.PROD.OUTLOOK.COM ([fe80::4e3:4ba3:b8aa:de93]) by TYCP286MB1969.JPNP286.PROD.OUTLOOK.COM ([fe80::4e3:4ba3:b8aa:de93%7]) with mapi id 15.20.5017.026; Sat, 26 Feb 2022 07:48:24 +0000 From: Paul Higgs To: FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH] AVS3: add support for AVS3 High profile - same syntax as AVS3 Main profile Thread-Index: AdgqbuEsGvztzk9NT6uYwaPZSpIHvQAch1sAAADygIA= Date: Sat, 26 Feb 2022 07:48:24 +0000 Message-ID: References: <20220226071804.GA16790@gmail.com> In-Reply-To: <20220226071804.GA16790@gmail.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-tmn: [Zy5eNEeBKIo2yyihzAiGToFrM/3blwjZ] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 5ca61591-95cb-47ce-6b32-08d9f8fc5e36 x-ms-traffictypediagnostic: TYCP286MB1106:EE_ x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: //yBAQm6obTLswPSRXbexHKmJHRNzbuayt/VGB8J0piO9PJTEllmsBRPVbnoAa/hV2jMkAVdb0w+d2T/o6tNu4I2W1B5leKTUCfSDB+aRyEcUiLWOwmi4LJyRy+L5Q1MSFtfPUppG37bwjYCCd/jeQ3dy1LRIg036ZxhPzBAJoZmNDcX0kxy8l0KwPWDmMvc9h9NG6BircJAMLkpUeP7J144QTT813UPwLzJK2CJa0LNgl5U11huPqvFKhnCOWxVBwC/wYtmhgJUf/eb9US5vm04nbtVc3gYDKlqC9TDgg66/sZ6inoORHrTQBCYJPKPXLv8YsiRO4o7SSiMriamOiNoidS63vDLI2SwX0Po6wgoEtiuwg3CPft8p/3n+6EDXNqQId3xW4xtbzgy2n/ICI8l/VMBrbkswNjGWSuNkP5dbiSltqhcfCXHxTyzYvN52nl2iyHE4JKmY9ENqBjuEFBRCIIVaN1ouzAx6dEkdqlICg1kF1r4T6IS3cF1wOOhB5Rumr+OYSfMW0f8Re+tn2nx98u4D61Ji8Z/CGy0kRTyGQKHBy/mwUQ5Qn23NTrc9+GMSchyluxjXZ8l+fTcgPNlEVPLLpW0F8tQ6FbDn1lizosDhOTttb+7ZnF0CTYO x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: tLxXPPLbyUbCVvENNYlt7P6jBcqaoyon1OD+VaHi9w5W3l4Dzqp4cR0A0ViK3DF+JPmCETblbsGiiUZy/no46RMux7i+62ckLQPCLYfX48+E6SIE4J9q6QRcM9ZRwYwyEFsYJVXZQqb6cOn7c62RK3ZdSLvLrmBx1mX+/maQUF4I7eqAxxQyWEMCM88sr/wJwKSace/TH8ycFGyj7b7HnkInVSCMeTEbaxUkiDQnNWLe3f3sQk8sJWk7yuQMM9KruR+utXIKvyVEXl6C1yH61YKpw6ewJ5MKEnIYkRrpRJqUquIkYNJWf1HEFD+KYGGJPPvnoTdYyffs9ZmA7+xoY54J/b2d12YN4lOPQHXxPIMtv05JcwygclckLGsOts3umk+E009+DIDtpMLggY3TOsJ//tPwgQ34Jj0pY+nFyeuB/s1Y3V7sc+T8OGwTallC2ZftY53dYlgOk3mwzcHrlqYmkLUkJcI4++aiA+gGIYJxcpcSzB64MGxbBCOHWRbT3VkujSbWXP5yeelbPd8u3+JqPe5elGy+nOvaIMMdkTyngJKRdnl0/102fGnwoJvFRxsPRPLEYuvfk48xuDvzVaUk0FKl75wVv+nmMWIEQ1wBR37PLk1HU1X1UXAcblyBwQmPRhJKT94Ix5eUyvVfsuLjTWaHPKnzwTq2LJpgmrDqPDXnCXI2Bt7l14AODaIDRlMWY3EuPoCair2tut1yTKFygWV0mVLwBtlALnEbJmHAZ0FeJjWW3lF+0BxPAiqkVtsVk2V9QBXM44LWl8XC/p8nvDvzb4dzz5MY4zBK+vlHrkqtGvTOcc1wFiYZsHBaYb2J3UT3JggkKdHwvrYjv/CGUEQZS+gjengW7WAbsXjF3AbpBEMgQ8BiCf29p9iEjYYrpz6TUwiCwjeYwima/ZxmXLoQW8I2mFEb6q/o6SX5+79GDp5TvcdGvQjG6Tcj MIME-Version: 1.0 X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-05f45.templateTenant X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: TYCP286MB1969.JPNP286.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-Network-Message-Id: 5ca61591-95cb-47ce-6b32-08d9f8fc5e36 X-MS-Exchange-CrossTenant-originalarrivaltime: 26 Feb 2022 07:48:24.4769 (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: TYCP286MB1106 Subject: Re: [FFmpeg-devel] [PATCH] AVS3: add support for AVS3 High profile - same syntax as AVS3 Main profile 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 > lance.lmwang@gmail.com > Sent: 26 February 2022 07:18 > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH] AVS3: add support for AVS3 High profile > - same syntax as AVS3 Main profile > > On Sat, Feb 26, 2022 at 05:50:39AM +0000, Paul Higgs wrote: > > This patch adds high level syntax support for parsing AVS3 High profile > bitstreams. > > Latest AVS3 specification including High profile is available at > > http://www.avs.org.cn/AVS3_download/en_index.asp > > > > Signed-off-by: Paul Higgs > > --- > > libavcodec/avs3.h | 2 ++ > > libavcodec/avs3_parser.c | 8 ++++---- > > libavformat/avs3dec.c | 2 +- > > 3 files changed, 7 insertions(+), 5 deletions(-) > > > > diff --git a/libavcodec/avs3.h b/libavcodec/avs3.h index > > 4189d9b583..c8caa58b0a 100644 > > --- a/libavcodec/avs3.h > > +++ b/libavcodec/avs3.h > > @@ -35,6 +35,8 @@ > > #define AVS3_FIRST_SLICE_START_CODE 0x00 > > #define AVS3_PROFILE_BASELINE_MAIN 0x20 > > #define AVS3_PROFILE_BASELINE_MAIN10 0x22 > > +#define AVS3_PROFILE_BASELINE_HIGH 0x30 > > +#define AVS3_PROFILE_BASELINE_HIGH10 0x32 > > > > #define AVS3_ISPIC(x) ((x) == AVS3_INTRA_PIC_START_CODE || (x) == > > AVS3_INTER_PIC_START_CODE) #define AVS3_ISUNIT(x) ((x) == > > AVS3_SEQ_START_CODE || AVS3_ISPIC(x)) diff --git > > a/libavcodec/avs3_parser.c b/libavcodec/avs3_parser.c index > > d04d96a03a..483b38fe76 100644 > > --- a/libavcodec/avs3_parser.c > > +++ b/libavcodec/avs3_parser.c > > @@ -91,11 +91,11 @@ static void > parse_avs3_nal_units(AVCodecParserContext *s, const uint8_t *buf, > > // sampe_precision(3) > > skip_bits(&gb, 47); > > > > - if (profile == AVS3_PROFILE_BASELINE_MAIN10) { > > - int sample_precision = get_bits(&gb, 3); > > - if (sample_precision == 1) { > > + if (profile == AVS3_PROFILE_BASELINE_MAIN10 || profile == > AVS3_PROFILE_BASELINE_HIGH10) { > > + int encoding_precision = get_bits(&gb, 3); > > + if (encoding_precision == 1) { > > Please keep sample_precision name, it's unrelated cosmetic change in this > patch. > Changed this because the three bits read in get_bits(&gb,3) are for the encoding precision. sample_precision syntax element was Skipped previously > > avctx->pix_fmt = AV_PIX_FMT_YUV420P; > > - } else if (sample_precision == 2) { > > + } else if (encoding_precision == 2) { > > avctx->pix_fmt = AV_PIX_FMT_YUV420P10LE; > > } else { > > avctx->pix_fmt = AV_PIX_FMT_NONE; diff --git > > a/libavformat/avs3dec.c b/libavformat/avs3dec.c index > > 2395df171b..335b5409f5 100644 > > --- a/libavformat/avs3dec.c > > +++ b/libavformat/avs3dec.c > > @@ -47,7 +47,7 @@ static int avs3video_probe(const AVProbeData *p) > > } > > if (state == AVS3_SEQ_START_CODE) { > > seq++; > > - if (*ptr != AVS3_PROFILE_BASELINE_MAIN && *ptr != > AVS3_PROFILE_BASELINE_MAIN10) > > + if (*ptr != AVS3_PROFILE_BASELINE_MAIN && *ptr != > > + AVS3_PROFILE_BASELINE_MAIN10 && *ptr != > AVS3_PROFILE_BASELINE_HIGH > > + && *ptr != AVS3_PROFILE_BASELINE_HIGH10) > > The line is too long. Ok, will wrap before 80 characters > > > return 0; > > } else if (AVS3_ISPIC(state)) { > > pic++; > > -- > > 2.30.0.windows.2 > > > > _______________________________________________ > > 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". > > -- > Thanks, > Limin Wang > _______________________________________________ > 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". _______________________________________________ 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".