commit bd3577ef5bae7d65e54bddc04f90b619330edb89
author Pablo Ceballos <pceballos@google.com> Mon Jun 20 17:40:34 2016 -0700
committer Pablo Ceballos <pceballos@google.com> Thu Jul 14 08:43:20 2016 -0700
tree e39428a6d84d9b4e3cedc63c2a99f45fe2002614
parent ab708f69952e8672d3f7e235b6d46d89f53b340e

Make sure constructors initialize all members

- Add default constructors to Parcelables that didn't have them and
  make sure all members are initialized by the constructors.
- Add missing fields to BufferItem flatten/unflatten.

Change-Id: I9aa83b326dc3aee757762e63be499dc96a74df91

services/surfaceflinger/DisplayHardware/HWC2On1Adapter.cpp

diff --git a/services/surfaceflinger/DisplayHardware/HWC2On1Adapter.cpp b/services/surfaceflinger/DisplayHardware/HWC2On1Adapter.cpp
index 0528d7b..cbe32e6 100644
--- a/services/surfaceflinger/DisplayHardware/HWC2On1Adapter.cpp
+++ b/services/surfaceflinger/DisplayHardware/HWC2On1Adapter.cpp
@@ -562,6 +562,7 @@
     mHwc1Id(-1),
     mConfigs(),
     mActiveConfig(nullptr),
+    mActiveColorMode(-1),
     mName(),
     mType(type),
     mPowerMode(PowerMode::Off),

services/surfaceflinger/DisplayHardware/HWC2On1Adapter.h

diff --git a/services/surfaceflinger/DisplayHardware/HWC2On1Adapter.h b/services/surfaceflinger/DisplayHardware/HWC2On1Adapter.h
index dc7c355..b46556d 100644
--- a/services/surfaceflinger/DisplayHardware/HWC2On1Adapter.h
+++ b/services/surfaceflinger/DisplayHardware/HWC2On1Adapter.h
@@ -247,6 +247,7 @@
                 public:
                     Config(Display& display)
                       : mDisplay(display),
+                        mId(0),
                         mAttributes() {}
 
                     bool isOnDisplay(const Display& display) const {