Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c5bac3aa authored by Glenn Kasten's avatar Glenn Kasten
Browse files

Fix build warnings and make build warnings be errors 

Test: screenrecord builds without warnings, both now and in future
Change-Id: Id8098c2be445715e0b4dd35e2b7aed0c7757d1fe
parent 0012417d

cmds/screenrecord/Android.mk

+1 −0
Original line number Diff line number Diff line
@@ -34,6 +34,7 @@ LOCAL_C_INCLUDES := \ 
	$(TOP)/frameworks/native/include/media/openmax \

	external/jpeg



LOCAL_CFLAGS := -Werror -Wall

LOCAL_CFLAGS += -Wno-multichar

#LOCAL_CFLAGS += -UNDEBUG

cmds/screenrecord/Overlay.cpp

+0 −1
Original line number Diff line number Diff line
@@ -203,7 +203,6 @@ void Overlay::processFrame_l() { 
    mGlConsumer->getTransformMatrix(texMatrix);

    nsecs_t monotonicNsec = mGlConsumer->getTimestamp();

    nsecs_t frameNumber = mGlConsumer->getFrameNumber();

    int64_t droppedFrames = 0;



    if (mLastFrameNumber > 0) {

        mTotalDroppedFrames += size_t(frameNumber - mLastFrameNumber) - 1;

cmds/screenrecord/screenrecord.cpp

+0 −1
Original line number Diff line number Diff line
@@ -215,7 +215,6 @@ static status_t prepareEncoder(float displayFps, sp<MediaCodec>* pCodec, 
 */

static status_t setDisplayProjection(const sp<IBinder>& dpy,

        const DisplayInfo& mainDpyInfo) {

    status_t err;



    // Set the region of the layer stack we're interested in, which in our

    // case is "all of it".  If the app is rotated (so that the width of the