diff options
author | Roman Stratiienko <roman.o.stratiienko@globallogic.com> | 2021-12-30 18:05:27 +0200 |
---|---|---|
committer | Roman Stratiienko <roman.o.stratiienko@globallogic.com> | 2022-01-12 10:33:55 +0200 |
commit | 753d107896e2129d255e1c7f5fdaf6f25251dd15 (patch) | |
tree | f30bfab94deece49fb6b8e92167f995671551769 /compositor/DrmDisplayComposition.h | |
parent | fc014f5792053a3a5712d1f6f65d55994dce24f4 (diff) | |
download | drm_hwcomposer-753d107896e2129d255e1c7f5fdaf6f25251dd15.tar.gz |
drm_hwcomposer: Make Planner class fully static
clang-tidy suggested to make all methods of Planner static,
since they don't use any instance members.
This makes creation of Planner instances unnecessary, therefore
remove all such cases.
Signed-off-by: Roman Stratiienko <roman.o.stratiienko@globallogic.com>
Diffstat (limited to 'compositor/DrmDisplayComposition.h')
-rw-r--r-- | compositor/DrmDisplayComposition.h | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/compositor/DrmDisplayComposition.h b/compositor/DrmDisplayComposition.h index d077aee..dcfd96e 100644 --- a/compositor/DrmDisplayComposition.h +++ b/compositor/DrmDisplayComposition.h @@ -30,7 +30,6 @@ namespace android { class Importer; -class Planner; constexpr size_t kUndefinedSourceLayer = UINT16_MAX; @@ -59,7 +58,7 @@ class DrmCompositionPlane { class DrmDisplayComposition { public: DrmDisplayComposition(const DrmDisplayComposition &) = delete; - DrmDisplayComposition(DrmCrtc *crtc, Planner *planner); + explicit DrmDisplayComposition(DrmCrtc *crtc); ~DrmDisplayComposition() = default; int SetLayers(DrmHwcLayer *layers, size_t num_layers); @@ -80,13 +79,8 @@ class DrmDisplayComposition { return crtc_; } - Planner *planner() const { - return planner_; - } - private: DrmCrtc *crtc_ = nullptr; - Planner *planner_ = nullptr; std::vector<DrmHwcLayer> layers_; std::vector<DrmCompositionPlane> composition_planes_; |