From 203339a2aa88bb31576233220d7ced73920a596f Mon Sep 17 00:00:00 2001 From: Ryan Harrison Date: Thu, 23 Aug 2018 20:58:14 +0000 Subject: 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 Reviewed-by: Tom Sepez Reviewed-by: Henrique Nakashima --- xfa/fxfa/parser/cxfa_node.cpp | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) (limited to 'xfa/fxfa/parser/cxfa_node.cpp') 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 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 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 { -- cgit v1.2.3