Differential D7893

Remove unnecessary vector allocation
ClosedPublic
Actions

Authored by poke1024 on Sep 20 2017, 5:36 PM.

Details

Reviewers
dkazakov
Group Reviewers
Krita
Commits
R37:90391bb3c8fc: Remove unnecessary vector allocation
R37:659420fc646e: Remove unnecessary vector allocation
Summary

Vector allocated but not used here. Shows up in profiling on OS X with about 0.5% (actual calls to new and free).

Diff Detail

Repository
R37 Krita
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
poke1024 created this revision.Sep 20 2017, 5:36 PM
dkazakov accepted this revision.Sep 22 2017, 9:38 PM
dkazakov added a subscriber: dkazakov.

Yep, seem to be forgotten during the refactoring.

This revision is now accepted and ready to land.Sep 22 2017, 9:38 PM
Closed by commit R37:90391bb3c8fc: Remove unnecessary vector allocation (authored by poke1024, committed by dkazakov). · Explain WhySep 25 2017, 12:35 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 19892

View Options

plugins/paintops/libpaintop/kis_curve_option.cpp

Loading...