From 550675ccffb7023e73022cdf411551cd62047c4b Mon Sep 17 00:00:00 2001 From: John Reck Date: Fri, 25 Oct 2024 10:07:45 -0400 Subject: [PATCH] Fix size of Rect fields in CROP spec The CROP spec specified to encode the rect as 4x int64, however Rect, gralloc4's libgralloctypes, and mapper5's IMapperMetadataTypes all used int32 for the Rect fields when encoding the crop since that's the actual size of the Rect's fields. Since VtsHalGraphicsMapperV4_0TargetTest's CROP test made use of libgralloctypes, fix the size in the spec to align with the implementations. Fixes: 359690632 Test: N/A Flag: EXEMPT doc only change Change-Id: If829c31188bea0f203d6ea6a96695414e9513256 --- .../android/hardware/graphics/common/StandardMetadataType.aidl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/graphics/common/aidl/android/hardware/graphics/common/StandardMetadataType.aidl b/graphics/common/aidl/android/hardware/graphics/common/StandardMetadataType.aidl index 8cfdae677c..e4da8904d1 100644 --- a/graphics/common/aidl/android/hardware/graphics/common/StandardMetadataType.aidl +++ b/graphics/common/aidl/android/hardware/graphics/common/StandardMetadataType.aidl @@ -282,7 +282,7 @@ enum StandardMetadataType { * When it is encoded into a byte stream, the total number of Rects is written using * 8 bytes in little endian. It is followed by each Rect. * - * To encode a Rect, write the following fields in this order each as 8 bytes in little endian: + * To encode a Rect, write the following fields in this order each as 4 bytes in little endian: * left, top, right and bottom. */ CROP = 16,