commit 617bb20364528b2d61550361ee580261c11faddb
author Chih-Hung Hsieh <chh@google.com> Wed Aug 22 14:45:27 2018 -0700
committer Chih-Hung Hsieh <chh@google.com> Wed Aug 22 14:45:27 2018 -0700
tree 39d8b272176ee1dc6a9f3d2c6838a6c60ada9879
parent 071d25dd3bdcce6a93d25aa927ca0c78016755d0

Fix undefined names used in ALOGV_IF.

* New ALOGV_IF macros and clang-tidy can now detect errors in ALOGV_IF.

Test: build with WITH_TIDY=1
Change-Id: If87e41e86bd5fc2d5ef0666026781688ea22db94

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 0181664..002b60d 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -1996,8 +1996,8 @@
     //   emit any black frames until a layer is added to the layer stack.
     bool mustRecompose = dirty && !(empty && wasEmpty);
 
-    ALOGV_IF(displayDevice->getDisplayType() == DisplayDevice::DISPLAY_VIRTUAL,
-            "dpy[%zu]: %s composition (%sdirty %sempty %swasEmpty)", dpy,
+    ALOGV_IF(display->getDisplayType() == DisplayDevice::DISPLAY_VIRTUAL,
+            "id[%d]: %s composition (%sdirty %sempty %swasEmpty)", display->getId(),
             mustRecompose ? "doing" : "skipping",
             dirty ? "+" : "-",
             empty ? "+" : "-",