diff options
| author | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-30 17:29:27 +0000 |
|---|---|---|
| committer | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-30 17:29:27 +0000 |
| commit | 40350b13714bed9c76d8bb0991c490b2ee14fe05 (patch) | |
| tree | ab2c88b3f9f5ba6e179a4547fb6b47bd3a768e95 /base/process_util_posix.cc | |
| parent | 3450c47c2b62b3db03c3d3a873252072baecd40c (diff) | |
| download | chromium_src-40350b13714bed9c76d8bb0991c490b2ee14fe05.zip chromium_src-40350b13714bed9c76d8bb0991c490b2ee14fe05.tar.gz chromium_src-40350b13714bed9c76d8bb0991c490b2ee14fe05.tar.bz2 | |
Minor C++ fixes found by Clang.
In cases where I've added an #include, it's generally due to Clang being
more picky about templates being available during expansions.
Review URL: http://codereview.chromium.org/1432003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43098 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/process_util_posix.cc')
| -rw-r--r-- | base/process_util_posix.cc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/base/process_util_posix.cc b/base/process_util_posix.cc index 25428fd..ec5f06c 100644 --- a/base/process_util_posix.cc +++ b/base/process_util_posix.cc @@ -221,12 +221,12 @@ void CloseSuperfluousFds(const base::InjectiveMultimap& saved_mapping) { const int fd = static_cast<int>(i); if (fd == STDIN_FILENO || fd == STDOUT_FILENO || fd == STDERR_FILENO) continue; - InjectiveMultimap::const_iterator i; - for (i = saved_mapping.begin(); i != saved_mapping.end(); i++) { - if (fd == i->dest) + InjectiveMultimap::const_iterator j; + for (j = saved_mapping.begin(); j != saved_mapping.end(); j++) { + if (fd == j->dest) break; } - if (i != saved_mapping.end()) + if (j != saved_mapping.end()) continue; // Since we're just trying to close anything we can find, |