Open-Harbor/osrm-backend
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 3 Wiki Activity

make osm_elements a loop variable, not a local variable

Browse Source 
The vector osm_elements was created and destroyed in each loop
iteration. It can be kept and cleared instead.
This commit is contained in:
Moritz Seemann 2017-01-11 16:56:42 +01:00
parent f12fdeb23e
commit 59ad7dddb2
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/extractor/extractor.cpp
View File

@ -173,10 +173,11 @@ int Extractor::run(ScriptingEnvironment &scripting_environment)
// setup restriction parser // setup restriction parser
const RestrictionParser restriction_parser(scripting_environment); const RestrictionParser restriction_parser(scripting_environment);
while (const osmium::memory::Buffer buffer = reader.read())
{
// create a vector of iterators into the buffer // create a vector of iterators into the buffer
std::vector<osmium::memory::Buffer::const_iterator> osm_elements; for (std::vector<osmium::memory::Buffer::const_iterator> osm_elements;
const osmium::memory::Buffer buffer = reader.read();
osm_elements.clear())
{
for (auto iter = std::begin(buffer), end = std::end(buffer); iter != end; ++iter) for (auto iter = std::begin(buffer), end = std::end(buffer); iter != end; ++iter)
{ {
osm_elements.push_back(iter); osm_elements.push_back(iter);