Differential D18208

Use reserve on QVector<Phase2Data>
ClosedPublic
Actions

Authored by graesslin on Jan 12 2019, 1:40 PM.

Details

Reviewers
zzag
Group Reviewers
KWin
Commits
R108:33c7edaaf479: Use reserve on QVector<Phase2Data>
Summary

We know that the maximum size is the number of windows in stacking
order. It might be that some are skipped as they are not ready for
painting, but overall it's better to reserve a little bit too much
memory than to have a growing QVector.

Test Plan

Compiles

Diff Detail

Repository
R108 KWin
Branch
phase2-reserve
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 7010
Build 7028: arc lint + arc unit
graesslin created this revision.Jan 12 2019, 1:40 PM
Restricted Application added a project: KWin. · View Herald TranscriptJan 12 2019, 1:40 PM
Restricted Application added a subscriber: kwin. · View Herald Transcript
graesslin requested review of this revision.Jan 12 2019, 1:40 PM
zzag accepted this revision.Jan 12 2019, 2:00 PM
This revision is now accepted and ready to land.Jan 12 2019, 2:00 PM
Closed by commit R108:33c7edaaf479: Use reserve on QVector<Phase2Data> (authored by graesslin). · Explain WhyJan 12 2019, 2:16 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mscene.cpp (2 lines)

Diff 49326

View Options

scene.cpp

Loading...