diff options
author | Ryan Harrison <rharrison@chromium.org> | 2018-06-01 15:40:13 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-06-01 15:40:13 +0000 |
commit | 44b29d13bffa30733ca0e780c71ca867a3e9a033 (patch) | |
tree | 0be335b4a2e8feae65c6c6f023d9fd1db0c9ee02 | |
parent | d60d7cb936ea50a0dfab26372f3a8958f84250e7 (diff) | |
download | pdfium-44b29d13bffa30733ca0e780c71ca867a3e9a033.tar.xz |
Remove unneeded call to GetNextSiblingWidget
The value is stored in a local scope variable at the end of a loop, so
is never used. The method is const, so is sideffect free, thus this
line has no overall effect.
Issue found with Clang Static Analyzer.
Change-Id: Ib20c5c121c8464433c6ad6d8e1439b929033d6d6
Reviewed-on: https://pdfium-review.googlesource.com/33530
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
-rw-r--r-- | xfa/fwl/cfwl_widgetmgr.cpp | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/xfa/fwl/cfwl_widgetmgr.cpp b/xfa/fwl/cfwl_widgetmgr.cpp index 7bae450374..f19479e72b 100644 --- a/xfa/fwl/cfwl_widgetmgr.cpp +++ b/xfa/fwl/cfwl_widgetmgr.cpp @@ -429,7 +429,6 @@ void CFWL_WidgetMgr::DrawChild(CFWL_Widget* parent, pDelegate->OnDrawWidget(pGraphics, widgetMatrix); DrawChild(child, clipBounds, pGraphics, &widgetMatrix); - child = GetNextSiblingWidget(child); } } |