Ben Woosley 9b72c988a0
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-
2018-06-11 13:12:55 -07:00
..
2018-02-16 08:59:28 -05:00
2017-09-29 16:02:39 +02:00
2018-02-10 09:55:54 -05:00
2018-03-06 21:52:53 +01:00
2018-06-05 02:16:24 -07:00
2018-03-21 11:57:57 +08:00
2018-06-04 14:55:00 -04:00
2018-05-06 13:55:33 +02:00
2018-02-11 10:48:15 +01:00
2018-03-06 19:52:19 +00:00
2018-03-06 19:52:19 +00:00
2018-03-13 17:04:31 -07:00
2018-04-16 13:24:14 -07:00
2018-03-13 17:04:31 -07:00
2018-05-09 12:21:05 -04:00
2018-05-19 11:16:00 -04:00
2018-04-16 13:24:14 -07:00
2018-04-16 13:24:14 -07:00
2018-03-21 08:34:44 +02:00
2018-05-30 16:02:09 -04:00
2018-03-06 20:28:08 -08:00