diff options
author | Tom Sepez <tsepez@chromium.org> | 2015-07-16 11:09:12 -0700 |
---|---|---|
committer | Tom Sepez <tsepez@chromium.org> | 2015-07-16 11:09:12 -0700 |
commit | fb07e2843dad0774d5842c2b08e7792164efc14a (patch) | |
tree | 4583acca1bdcf5b60fad79af0b9fa405f192c5d8 /OWNERS | |
parent | cc8b4d8763151f51c9fe71ce9b64b2a0d7ddcc88 (diff) | |
download | pdfium-fb07e2843dad0774d5842c2b08e7792164efc14a.tar.xz |
Tidy up CPDFDOC_Environment.
- untabify as encountered.
- Only put single-statement method in .h file, move more
complex methods to .cpp (counting an if without braces as
a single statement, killing braces as needed).
- Move invariant arguments to constructor and make
corresponding members const.
- Make all members private and add accessor methods.
- Make existing accessor methods const where possible.
- Kill meaningless asserts.
- Add helper functions in place of duplicate code.
- Rename GetCurrentDoc() to GetSDKDocument(), since the class
has two document members, one of CPDF_Document and one of
CPDFSDK_Document, making it clear which one you get.
- Simplify some logic with early returns.
R=thestig@chromium.org
Review URL: https://codereview.chromium.org/1235393002 .
Diffstat (limited to 'OWNERS')
0 files changed, 0 insertions, 0 deletions