diff options
| author | 2017-12-05 23:01:28 +0000 | |
|---|---|---|
| committer | 2017-12-05 23:01:28 +0000 | |
| commit | c8889e310cd3721a93ca0f7df1a03d4330946278 (patch) | |
| tree | 358f739d9b08fa25705c8393b3ff511f713e1d17 | |
| parent | 069ac6a5dd9af2ee596efb628988520fbdd4a001 (diff) | |
| parent | 30a2d0531aebdc244aa1b0f7fcfcff9d965967cb (diff) | |
Merge "Use -Werror in frameworks/native/libs/vr/libbufferhubqueue"
| -rw-r--r-- | libs/vr/libbufferhubqueue/tests/Android.bp | 2 | ||||
| -rw-r--r-- | libs/vr/libbufferhubqueue/tests/buffer_transport_benchmark.cpp | 7 |
2 files changed, 4 insertions, 5 deletions
diff --git a/libs/vr/libbufferhubqueue/tests/Android.bp b/libs/vr/libbufferhubqueue/tests/Android.bp index abeb71922e..5eba913ac3 100644 --- a/libs/vr/libbufferhubqueue/tests/Android.bp +++ b/libs/vr/libbufferhubqueue/tests/Android.bp @@ -68,6 +68,8 @@ cc_benchmark { "-DLOG_TAG=\"buffer_transport_benchmark\"", "-DTRACE=0", "-O2", + "-Wall", + "-Werror", ], name: "buffer_transport_benchmark", tags: ["optional"], diff --git a/libs/vr/libbufferhubqueue/tests/buffer_transport_benchmark.cpp b/libs/vr/libbufferhubqueue/tests/buffer_transport_benchmark.cpp index 658b496ecd..d4d25b0013 100644 --- a/libs/vr/libbufferhubqueue/tests/buffer_transport_benchmark.cpp +++ b/libs/vr/libbufferhubqueue/tests/buffer_transport_benchmark.cpp @@ -34,7 +34,6 @@ using ::benchmark::State; static const String16 kBinderService = String16("bufferTransport"); static const uint32_t kBufferWidth = 100; static const uint32_t kBufferHeight = 1; -static const uint32_t kBufferLayerCount = 1; static const uint32_t kBufferFormat = HAL_PIXEL_FORMAT_BLOB; static const uint64_t kBufferUsage = GRALLOC_USAGE_SW_READ_OFTEN | GRALLOC_USAGE_SW_WRITE_OFTEN; @@ -75,10 +74,9 @@ class BufferTransportService : public BBinder { private: struct FrameListener : public ConsumerBase::FrameAvailableListener { public: - FrameListener(BufferTransportService* service, + FrameListener(BufferTransportService* /*service*/, sp<BufferItemConsumer> buffer_item_consumer) - : service_(service), - buffer_item_consumer_(buffer_item_consumer) {} + : buffer_item_consumer_(buffer_item_consumer) {} void onFrameAvailable(const BufferItem& /*item*/) override { BufferItem buffer; @@ -106,7 +104,6 @@ class BufferTransportService : public BBinder { } private: - BufferTransportService* service_ = nullptr; sp<BufferItemConsumer> buffer_item_consumer_; }; |