Differential D20251

Drop unnecessary PaintUtils::containingRect()
ClosedPublic
Actions

Authored by volkov on Apr 4 2019, 3:12 PM.

Details

Reviewers
ngraham
Group Reviewers
Gwenview
Commits
R260:b6e6ec6e92d0: Drop unnecessary PaintUtils::containingRect()
Summary

It's equal to QRectF::toAlignedRect().

Diff Detail

Repository
R260 Gwenview
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 10611
Build 10629: arc lint + arc unit
volkov created this revision.Apr 4 2019, 3:12 PM
Restricted Application added a project: Gwenview. · View Herald TranscriptApr 4 2019, 3:12 PM
volkov requested review of this revision.Apr 4 2019, 3:12 PM
volkov updated this revision to Diff 55732.Apr 8 2019, 11:22 AM

re-target to master

ngraham accepted this revision.Apr 8 2019, 1:33 PM
ngraham added a subscriber: ngraham.

Nice catch!

This revision is now accepted and ready to land.Apr 8 2019, 1:33 PM
volkov closed this revision.Apr 8 2019, 5:12 PM

Revision Contents
Changeset List

Diff 55732

View Options

lib/imagescaler.cpp

Loading...
View Options

lib/paintutils.h

Loading...
View Options

lib/paintutils.cpp

Loading...
View Options

lib/redeyereduction/redeyereductionimageoperation.cpp

Loading...
View Options

lib/redeyereduction/redeyereductiontool.cpp

Loading...
View Options

tests/auto/CMakeLists.txt

Loading...
View Options

tests/auto/paintutilstest.h

Loading...
View Options

tests/auto/paintutilstest.cpp

Loading...