Skip to content

Commit aacae49

Browse files
Anton IvanovAndroid (Google) Code Review
authored andcommitted
Merge "Fix some clang warnings in SurfaceFlinger.{h, cpp}" into main
2 parents 978e6f3 + 2417b9f commit aacae49

2 files changed

Lines changed: 2 additions & 7 deletions

File tree

services/surfaceflinger/SurfaceFlinger.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7907,9 +7907,8 @@ status_t SurfaceFlinger::applyRefreshRateSelectorPolicy(
79077907
const scheduler::RefreshRateSelector::Policy currentPolicy = selector.getCurrentPolicy();
79087908
ALOGV("Setting desired display mode specs: %s", currentPolicy.toString().c_str());
79097909

7910-
if (const bool isPacesetter =
7911-
mScheduler->onDisplayModeChanged(displayId, selector.getActiveMode(),
7912-
/*clearContentRequirements*/ true)) {
7910+
if (mScheduler->onDisplayModeChanged(displayId, selector.getActiveMode(),
7911+
/*clearContentRequirements*/ true)) {
79137912
mDisplayModeController.updateKernelIdleTimer(displayId);
79147913
}
79157914

services/surfaceflinger/SurfaceFlinger.h

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,6 @@
7070

7171
#include <common/FlagManager.h>
7272
#include "ActivePictureTracker.h"
73-
#include "BackgroundExecutor.h"
7473
#include "Display/DisplayModeController.h"
7574
#include "Display/PhysicalDisplay.h"
7675
#include "Display/VirtualDisplaySnapshot.h"
@@ -102,12 +101,9 @@
102101
#include <atomic>
103102
#include <cstdint>
104103
#include <functional>
105-
#include <map>
106104
#include <memory>
107105
#include <mutex>
108106
#include <optional>
109-
#include <queue>
110-
#include <set>
111107
#include <string>
112108
#include <thread>
113109
#include <type_traits>

0 commit comments

Comments
 (0)