From 9cc81afb477c5c16c857831672e4090f878aa616 Mon Sep 17 00:00:00 2001 From: Gerrit User 1115898 <1115898@3ce6091f-6c88-37e8-8c75-72f92ae8dfba> Date: Thu, 12 Dec 2024 01:48:20 -0800 Subject: [PATCH] Update patch set 2 Patch Set 2: Code-Review+1 (1 comment) Patch-set: 2 Reviewer: Gerrit User 1115898 <1115898@3ce6091f-6c88-37e8-8c75-72f92ae8dfba> Label: Code-Review=+1, 4163703ccb2ea735fe96186ca667409fb46405e0 Attention: {"person_ident":"Gerrit User 3775295 \u003c3775295@3ce6091f-6c88-37e8-8c75-72f92ae8dfba\u003e","operation":"ADD","reason":"\u003cGERRIT_ACCOUNT_1115898\u003e replied on the change"} --- dcbf20c24c28b9b662fd44f2e5a5165e3d4e0789 | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/dcbf20c24c28b9b662fd44f2e5a5165e3d4e0789 b/dcbf20c24c28b9b662fd44f2e5a5165e3d4e0789 index 42add73b6..8b62e89ed 100644 --- a/dcbf20c24c28b9b662fd44f2e5a5165e3d4e0789 +++ b/dcbf20c24c28b9b662fd44f2e5a5165e3d4e0789 @@ -16,6 +16,23 @@ "message": "Hi, I am a developer of Tencent. Our project requires scaling for the NV24 format, but the function is currently missing in scale.h. This commit adds the necessary implementation to support NV24 scaling.", "revId": "dcbf20c24c28b9b662fd44f2e5a5165e3d4e0789", "serverId": "3ce6091f-6c88-37e8-8c75-72f92ae8dfba" + }, + { + "unresolved": false, + "key": { + "uuid": "8c849ee2_236a62c4", + "filename": "/PATCHSET_LEVEL", + "patchSetId": 2 + }, + "lineNbr": 0, + "author": { + "id": 1115898 + }, + "writtenOn": "2024-12-12T09:48:20Z", + "side": 1, + "message": "Thanks for the contribution! Looks good. Would be nice to have a unit test, but really the planar functions need more complete tests.\nIn the future there is P010 that is equivalent to NV12 but 10 bit and P410 is like NV24 with 4:4:4 subsampling.\nAlso NV42 is the same as NV24 but VU vs UV, which doesnt matter, but you might provide a macro in the header", + "revId": "dcbf20c24c28b9b662fd44f2e5a5165e3d4e0789", + "serverId": "3ce6091f-6c88-37e8-8c75-72f92ae8dfba" } ] } \ No newline at end of file