aboutsummaryrefslogtreecommitdiff
path: root/COPYING
diff options
context:
space:
mode:
authorBen Woosley <ben.woosley@gmail.com>2018-05-15 15:41:53 -0700
committerBen Woosley <ben.woosley@gmail.com>2018-06-11 13:12:55 -0700
commit9b72c988a0050d8932275c74c60928918ee7ef71 (patch)
tree2853b851aaffa1fe04f3f904e094b2962941f120 /COPYING
parent7c32b414b6325743c99fed1208bc53ab0fa1996f (diff)
scripted-diff: Avoid temporary copies when looping over std::map
The ::value_type of the std::map/std::multimap/std::unordered_map containers is std::pair<const Key, T>. Dropping the const results in an unnecessary copy, for example in C++11 range-based loops. For this I started with a more general scripted diff, then narrowed it down based on the inspection showing that all actual map/multimap/unordered_map variables used in loops start with m or have map in the name. -BEGIN VERIFY SCRIPT- sed -i -E 's/for \(([^<]*)std::pair<([^c])(.+) : m/for (\1std::pair<const \2\3 : m/' src/*.cpp src/**/*.cpp sed -i -E 's/for \(([^<]*)std::pair<([^c])(.+) : (.*)map/for (\1std::pair<const \2\3 : \4map/' src/*.cpp src/**/*.cpp -END VERIFY SCRIPT-
Diffstat (limited to 'COPYING')
0 files changed, 0 insertions, 0 deletions