Reuse QString instances in parseAddrList

Review Request #106836 - Created Oct. 13, 2012 and submitted

Information
Milian Wolff
kdepim-runtime
master
Reviewers
kdepim, kdepimlibs
Leverage QString implicit sharing to reduce memory consumption. In my tests, the memory decreases roughly by ~20MB from a total of 300 down to about 280MB.

My question is rather, whether this really needs to be thread safe or not. Maybe also some performance testing is required?
Ran KMail, not noticeably slower but uses less ram. Nothing else seems to have changed.
Volker Krause
Milian Wolff
Commit Hook
Milian Wolff
Review request changed

Status: Closed (submitted)

Loading...