diff options
author | Ryan Harrison <rharrison@chromium.org> | 2018-08-23 20:58:14 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-08-23 20:58:14 +0000 |
commit | 203339a2aa88bb31576233220d7ced73920a596f (patch) | |
tree | 895fc5fcc07107c8cc6f81b9c6e0f12faef9fc1e /xfa/fxfa/parser/cxfa_node.cpp | |
parent | 3b45012d57884b06915eb5a1f54fbba04a45e807 (diff) | |
download | pdfium-203339a2aa88bb31576233220d7ced73920a596f.tar.xz |
Fix shadowed variables
This CL fixes instances of variable shadowing that are
discovered by turning on -Wshadow.
BUG=pdfium:1137
Change-Id: I418d50de89ecbeb12e85b23a358bc61e8f16e888
Reviewed-on: https://pdfium-review.googlesource.com/41150
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Diffstat (limited to 'xfa/fxfa/parser/cxfa_node.cpp')
-rw-r--r-- | xfa/fxfa/parser/cxfa_node.cpp | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/xfa/fxfa/parser/cxfa_node.cpp b/xfa/fxfa/parser/cxfa_node.cpp index c41a1ac97a..7e582c0518 100644 --- a/xfa/fxfa/parser/cxfa_node.cpp +++ b/xfa/fxfa/parser/cxfa_node.cpp @@ -3985,10 +3985,10 @@ void CXFA_Node::SetItemState(int32_t nIndex, int32_t iSel = -1; std::vector<WideString> wsValueArray = GetSelectedItemsValue(); - auto it = std::find(wsValueArray.begin(), wsValueArray.end(), - wsSaveTextArray[nIndex]); - if (it != wsValueArray.end()) - iSel = it - wsValueArray.begin(); + auto value_iter = std::find(wsValueArray.begin(), wsValueArray.end(), + wsSaveTextArray[nIndex]); + if (value_iter != wsValueArray.end()) + iSel = value_iter - wsValueArray.begin(); if (IsChoiceListMultiSelect()) { if (bSelected) { @@ -4003,9 +4003,10 @@ void CXFA_Node::SetItemState(int32_t nIndex, } } else if (iSel >= 0) { std::vector<int32_t> iSelArray = GetSelectedItems(); - auto it = std::find(iSelArray.begin(), iSelArray.end(), nIndex); - if (it != iSelArray.end()) - iSelArray.erase(it); + auto selected_iter = + std::find(iSelArray.begin(), iSelArray.end(), nIndex); + if (selected_iter != iSelArray.end()) + iSelArray.erase(selected_iter); SetSelectedItems(iSelArray, bNotify, bScriptModify, bSyncData); } } else { |