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

Commit a282f588 authored by Dan Stoza's avatar Dan Stoza
Browse files

SF: Fix warnings in tests and enable -Werror 

Fixes unused-parameter warnings in the vsync and waitforvsync tests,
and enables -Werror in their respective Android.mks.

Bug: 36678673
Test: mm -j, observe no compiler warnings/errors
Change-Id: I6e2c5d2d2937b045794a53d90aeab7653b42e6c5
parent e1a0c6db

services/surfaceflinger/tests/vsync/Android.mk

+2 −0
Original line number Diff line number Diff line
@@ -15,4 +15,6 @@ LOCAL_MODULE:= test-vsync-events 


LOCAL_MODULE_TAGS := tests



LOCAL_CFLAGS := -Werror



include $(BUILD_EXECUTABLE)

services/surfaceflinger/tests/vsync/vsync.cpp

+2 −2
Original line number Diff line number Diff line
@@ -20,7 +20,7 @@ 


using namespace android;



int receiver(int fd, int events, void* data)

int receiver(int /*fd*/, int /*events*/, void* data)

{

    DisplayEventReceiver* q = (DisplayEventReceiver*)data;
@@ -47,7 +47,7 @@ int receiver(int fd, int events, void* data) 
    return 1;

}



int main(int argc, char** argv)

int main(int /*argc*/, char** /*argv*/)

{

    DisplayEventReceiver myDisplayEvent;

services/surfaceflinger/tests/waitforvsync/Android.mk

+2 −0
Original line number Diff line number Diff line
@@ -11,4 +11,6 @@ LOCAL_MODULE:= test-waitforvsync 


LOCAL_MODULE_TAGS := tests



LOCAL_CFLAGS := -Werror



include $(BUILD_EXECUTABLE)

services/surfaceflinger/tests/waitforvsync/waitforvsync.cpp

+1 −1
Original line number Diff line number Diff line
@@ -29,7 +29,7 @@ 
#define FBIO_WAITFORVSYNC   _IOW('F', 0x20, __u32)

#endif



int main(int argc, char** argv) {

int main(int /*argc*/, char** /*argv*/) {

    int fd = open("/dev/graphics/fb0", O_RDWR);

    if (fd >= 0) {

        do {