tree 155785aa7bbd4b5d09e6d456a7c4e3ac4a8c6676
parent b1bd2eae67c333ce792e777359251a0f7a739670
author Tom Sepez <tsepez@chromium.org> 1622855067 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1622855067 +0000

[code health] Prefer Optional<>::value() to operator*()

Improves readability especially when the underlying type has its own
operator*() method. Rely on built-in operator==() between optional and
underlying type to avoid some calls to value(), but stop short of doing
the same for `>` and `<' comparisons.

Change-Id: I207c79c04e3f4013f07a423decb250a826261cb8
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/81352
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>