diff options
| author | Ryan Harrison <rharrison@chromium.org> | 2018-06-05 15:54:51 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-06-05 15:54:51 +0000 |
| commit | eaba5af84597164281ddd972a5e98631b18d783b (patch) | |
| tree | 4ae287d9093e3775b6f32b8ea9c41e3e2bbae5ad /fxjs | |
| parent | 6cec70acf4ed6839c4a865c41381a6ac37de97e8 (diff) | |
| download | pdfium-eaba5af84597164281ddd972a5e98631b18d783b.tar.xz | |
Clean up CFX_TxtBreak
Removed some unused assignments and now unused variables.
Also rewrote a for loop so that its secondary condition is now in the
control block. The static analyzer was claiming it was possible for a
division by 0, though following the control path it was claiming, I am
pretty sure it should have exited early. This reorganizing of the code
makes it a bit more readable and eliminates the div by 0 warning from
the static analyzer.
Issues found with Clang Static Analyzer.
Change-Id: I91f1b42374734665b06724e9fafa6490e06bd3da
Reviewed-on: https://pdfium-review.googlesource.com/33910
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: Ryan Harrison <rharrison@chromium.org>
Diffstat (limited to 'fxjs')
0 files changed, 0 insertions, 0 deletions