commit f815d38c2ad191b3cbe7e259b9ef2fc974875b72
author Roman Stratiienko <roman.o.stratiienko@globallogic.com> Thu Dec 30 19:23:14 2021 +0200
committer Roman Stratiienko <roman.o.stratiienko@globallogic.com> Wed Jan 12 10:47:23 2022 +0200
tree f5bf30af368d1aaf10a359f16fe31c0ebcded454
parent 57b9ba3a7dc78318184eec74a4a0bbce4d2ce759

drm_hwcomposer: Raise clang-tidy level of some files to NORMAL

Signed-off-by: Roman Stratiienko <roman.o.stratiienko@globallogic.com>

compositor/DrmDisplayCompositor.cpp

diff --git a/compositor/DrmDisplayCompositor.cpp b/compositor/DrmDisplayCompositor.cpp
index 2c2ec65..fc95b26 100644
--- a/compositor/DrmDisplayCompositor.cpp
+++ b/compositor/DrmDisplayCompositor.cpp
@@ -70,7 +70,7 @@
 auto DrmDisplayCompositor::CommitFrame(AtomicCommitArgs &args) -> int {
   ATRACE_CALL();
 
-  if (args.active && *args.active == active_frame_state.crtc_active_state) {
+  if (args.active && *args.active == active_frame_state_.crtc_active_state) {
     /* Don't set the same state twice */
     args.active.reset();
   }
@@ -80,7 +80,7 @@
     return 0;
   }
 
-  if (!active_frame_state.crtc_active_state) {
+  if (!active_frame_state_.crtc_active_state) {
     /* Force activate display */
     args.active = true;
   }
@@ -204,7 +204,7 @@
       connector->set_active_mode(*args.display_mode);
     }
 
-    active_frame_state = std::move(new_frame_state);
+    active_frame_state_ = std::move(new_frame_state);
 
     if (crtc->out_fence_ptr_property()) {
       args.out_fence = UniqueFd((int)out_fence);

compositor/DrmDisplayCompositor.h

diff --git a/compositor/DrmDisplayCompositor.h b/compositor/DrmDisplayCompositor.h
index f3169e8..a883f94 100644
--- a/compositor/DrmDisplayCompositor.h
+++ b/compositor/DrmDisplayCompositor.h
@@ -78,13 +78,13 @@
 
     /* To avoid setting the inactive state twice, which will fail the commit */
     bool crtc_active_state{};
-  } active_frame_state;
+  } active_frame_state_;
 
   auto NewFrameState() -> KmsState {
     return (KmsState){
-        .used_planes = active_frame_state.used_planes,
-        .used_framebuffers = active_frame_state.used_framebuffers,
-        .crtc_active_state = active_frame_state.crtc_active_state,
+        .used_planes = active_frame_state_.used_planes,
+        .used_framebuffers = active_frame_state_.used_framebuffers,
+        .crtc_active_state = active_frame_state_.crtc_active_state,
     };
   }