Avoid a lot allocations in KConfig
ClosedPublic

Authored by cullmann on Mar 5 2017, 3:56 PM.

Details

Summary

Avoid a lot allocations in KConfig: We anyway free this QByteArray direcly after use, no need to shrink it, which only leads (if at all) to unnecessary allocations.

Seen in e.g. heaptrack with Kate. (btw., thanks to Milian for writing that :)

Test Plan

Given it only alters the allocation behavior, no changes visible, beside less heap allocs.

Diff Detail

Repository
R237 KConfig
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
cullmann created this revision.Mar 5 2017, 3:56 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptMar 5 2017, 3:56 PM
dfaure accepted this revision.Mar 5 2017, 6:56 PM
This revision is now accepted and ready to land.Mar 5 2017, 6:56 PM
This revision was automatically updated to reflect the committed changes.