diff options
| author | Tatsuhiro Tsujikawa <tatsuhiro.t@gmail.com> | 2010-02-28 16:04:52 +0000 |
|---|---|---|
| committer | Tatsuhiro Tsujikawa <tatsuhiro.t@gmail.com> | 2010-02-28 16:04:52 +0000 |
| commit | 72e475dfdeaa4a87dff0eef994e7814e94c40363 (patch) | |
| tree | 94a84e90c14876db223e4a733f651fe0b034f5e9 /src/PStringBuildVisitor.cc | |
| parent | c342bde9627b434effbc0cd10b817670d1da0923 (diff) | |
2010-03-01 Tatsuhiro Tsujikawa <t-tujikawa@users.sourceforge.net>
Avoid to calculate end() iterator in each loop. std::deque is
particularly slow. Make sure that recalculate end iterator if
element is erased during loop.
Diffstat (limited to 'src/PStringBuildVisitor.cc')
| -rw-r--r-- | src/PStringBuildVisitor.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/PStringBuildVisitor.cc b/src/PStringBuildVisitor.cc index 81d1171a..599a0e86 100644 --- a/src/PStringBuildVisitor.cc +++ b/src/PStringBuildVisitor.cc @@ -75,8 +75,8 @@ void PStringBuildVisitor::visit(PStringNumLoop& s) void PStringBuildVisitor::visit(PStringSelect& s) { const std::vector<std::string>& values = s.getValues(); - for(std::vector<std::string>::const_iterator i = values.begin(); - i != values.end(); ++i) { + for(std::vector<std::string>::const_iterator i = values.begin(), + eoi = values.end(); i != eoi; ++i) { PStringSegment(*i, s.getNext()).accept(*this); } } |