Skip to content

Commit

Permalink
Explicitly initialise base class in copy constructor
Browse files Browse the repository at this point in the history
Fixes:
In file included from frameworks/av/media/libmedia/include/media/IMediaPlayer.h:27:0,
                 from frameworks/av/media/libmedia/include/media/IMediaPlayerService.h:29,
                 from frameworks/av/media/libmedia/include/media/IMediaDeathNotifier.h:21,
                 from hardware/qcom/display/msm8994/libhwcomposer/hwc_qclient.h:37,
                 from hardware/qcom/display/msm8994/libhwcomposer/hwc_utils.cpp:41:
frameworks/av/include/media/VolumeShaper.h: In copy constructor 'android::media::VolumeShaper::Configuration::Configuration(const
android::media::VolumeShaper::Configuration&)':
frameworks/av/include/media/VolumeShaper.h:135:18: error: base class 'class android::Parcelable' should be explicitly initialized in the copy constructor
[-Werror=extra]
         explicit Configuration(const Configuration &configuration)
                  ^
cc1plus: all warnings being treated as errors

Change-Id: I37b77d21bc625355309a634520153c2b9d6332c8
  • Loading branch information
harryyoud authored and Darkstar085 committed Oct 26, 2018
1 parent 3741232 commit 2ca5afc
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions include/media/VolumeShaper.h
Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,7 @@ class VolumeShaper {
Configuration()
: Interpolator<S, T>()
, RefBase()
, Parcelable()
, mType(TYPE_SCALE)
, mId(-1)
, mOptionFlags(OPTION_FLAG_NONE)
Expand Down

0 comments on commit 2ca5afc

Please sign in to comment.