From cd26147cd28716e87f4c5bd6cd427431d1e5fc38 Mon Sep 17 00:00:00 2001 From: Peiyong Lin Date: Tue, 6 Oct 2020 18:05:25 -0700 Subject: Remove hard code color space. In general when there's a color space involved, we must explicitly specify instead of hard coding. Bug: N/A Test: atest SurfaceFlinger_unittest Change-Id: I87c84990b4ec4bd815d7d552346c5fc8c7274135 --- libs/gui/LayerState.cpp | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'libs/gui/LayerState.cpp') diff --git a/libs/gui/LayerState.cpp b/libs/gui/LayerState.cpp index bde73bab2e..8594ab3bd5 100644 --- a/libs/gui/LayerState.cpp +++ b/libs/gui/LayerState.cpp @@ -592,19 +592,20 @@ status_t CaptureArgs::write(Parcel& output) const { SAFE_PARCEL(output.writeFloat, frameScale); SAFE_PARCEL(output.writeBool, captureSecureLayers); SAFE_PARCEL(output.writeInt32, uid); - SAFE_PARCEL(output.writeBool, useRGBColorSpace); + SAFE_PARCEL(output.writeInt32, static_cast(dataspace)); return NO_ERROR; } status_t CaptureArgs::read(const Parcel& input) { - int32_t format = 0; - SAFE_PARCEL(input.readInt32, &format); - pixelFormat = static_cast(format); + int32_t value = 0; + SAFE_PARCEL(input.readInt32, &value); + pixelFormat = static_cast(value); SAFE_PARCEL(input.read, sourceCrop); SAFE_PARCEL(input.readFloat, &frameScale); SAFE_PARCEL(input.readBool, &captureSecureLayers); SAFE_PARCEL(input.readInt32, &uid); - SAFE_PARCEL(input.readBool, &useRGBColorSpace); + SAFE_PARCEL(input.readInt32, &value); + dataspace = static_cast(value); return NO_ERROR; } -- cgit v1.2.3-59-g8ed1b