diff options
| author | Ben Woosley <[email protected]> | 2018-05-15 15:41:53 -0700 |
|---|---|---|
| committer | Ben Woosley <[email protected]> | 2018-06-11 13:12:55 -0700 |
| commit | 9b72c988a0050d8932275c74c60928918ee7ef71 (patch) | |
| tree | 2853b851aaffa1fe04f3f904e094b2962941f120 /src/httpserver.cpp | |
| parent | Merge #13230: Simplify include analysis by enforcing the developer guide's in... (diff) | |
| download | discoin-9b72c988a0050d8932275c74c60928918ee7ef71.tar.xz discoin-9b72c988a0050d8932275c74c60928918ee7ef71.zip | |
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 'src/httpserver.cpp')
0 files changed, 0 insertions, 0 deletions