tree 4ae287d9093e3775b6f32b8ea9c41e3e2bbae5ad
parent 6cec70acf4ed6839c4a865c41381a6ac37de97e8
author Ryan Harrison <rharrison@chromium.org> 1528214091 +0000
committer Chromium commit bot <commit-bot@chromium.org> 1528214091 +0000

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>