Introduce KDevelop::Stack (replaces QStack)
ClosedPublic

Authored by kfunk on Sep 6 2015, 4:56 PM.

Details

Summary

See class documentation for more details.

When using KDevelop::Stack instead of QStack the heap allocations go almost done to zero
within KDevelop. QStack::pop in particular is implemented *very*
inefficient, every call causes a realloc b/c QStack tries to squeeze the
container.

Diff Detail

Repository
R33 KDevPlatform
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kfunk updated this revision to Diff 765.Sep 6 2015, 4:56 PM
kfunk retitled this revision from to Introduce KDevelop::Stack (replaces QStack).
kfunk updated this object.
kfunk edited the test plan for this revision. (Show Details)
Restricted Application added a subscriber: kdevelop-devel. · View Herald TranscriptSep 6 2015, 4:56 PM
kfunk added a project: KDevelop.
kfunk updated this revision to Diff 769.Sep 6 2015, 5:03 PM

Fixed commit message

mwolff added a subscriber: mwolff.Sep 6 2015, 6:01 PM

lgtm in general! thanks

language/duchain/problem.h
31 ↗(On Diff #769)

remove, seems to be unused?

language/duchain/types/typeregister.h
120 ↗(On Diff #769)

just remove this?

language/util/setrepository.cpp
19 ↗(On Diff #769)

remove, seems to be unused

util/stack.h
53 ↗(On Diff #769)

please put these on multiple lines, following the KDev coding style

mwolff accepted this revision.Oct 8 2015, 10:09 AM
mwolff added a reviewer: mwolff.

accepting

This revision is now accepted and ready to land.Oct 8 2015, 10:09 AM
This revision was automatically updated to reflect the committed changes.