diff options
| author | 2017-08-03 15:48:25 -0700 | |
|---|---|---|
| committer | 2017-08-03 17:38:27 -0700 | |
| commit | cb057c2e6bf23b43df49282c0db26b663e4535f7 (patch) | |
| tree | 13c5977423159cd4e0be6bc3b2936c398b6004c7 /libs/gui/BufferQueueProducer.cpp | |
| parent | 10288989e36fee33e50e2a79bc6111b47cd729bb (diff) | |
Fix clang-tidy performance warnings in frameworks/native.
* Use const reference parameter type to avoid unnecessary copy.
* Use more efficient overloaded string methods.
* Use const reference type for loop index variables to avoid unnecessary copy.
Bug: 30407689
Bug: 30411878
Bug: 30413223
Test: build with WITH_TIDY=1
Change-Id: I75ed62e2cb9939878b8932ed512ceb7e241edd6f
Diffstat (limited to 'libs/gui/BufferQueueProducer.cpp')
| -rw-r--r-- | libs/gui/BufferQueueProducer.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libs/gui/BufferQueueProducer.cpp b/libs/gui/BufferQueueProducer.cpp index 3424012acf..bcfa7bccd7 100644 --- a/libs/gui/BufferQueueProducer.cpp +++ b/libs/gui/BufferQueueProducer.cpp @@ -764,7 +764,7 @@ status_t BufferQueueProducer::queueBuffer(int slot, input.deflate(&requestedPresentTimestamp, &isAutoTimestamp, &dataSpace, &crop, &scalingMode, &transform, &acquireFence, &stickyTransform, &getFrameTimestamps); - Region surfaceDamage = input.getSurfaceDamage(); + const Region& surfaceDamage = input.getSurfaceDamage(); if (acquireFence == NULL) { BQ_LOGE("queueBuffer: fence is NULL"); |