tree b9eac9d802b0c1b279320905ba1de9708e8ea095
parent 3efb74a25dcf9d5c6879af4b3b95b76d2009de47
author Tom Sepez <tsepez@chromium.org> 1626742893 +0000
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1626742893 +0000

Tidy FXSYS_DWORD_GET_MSBFIRST() and friends.

We'd thought about converting these to functions taking (const uint8_t*)
arguments, but it might be nice to apply these to anything supporting a
operator[] (such as span<>), which would mean either templates, or just
leaving them as macros.

-- Rename WORD/DWORD to UINT16/UINT32 to avoid those "word" words.
-- Parenthesize to allow e.g. GET(ptr + 10)
-- Perform unsigned shifts only (uint16_t silently promotes to int).
-- De-duplicate implementation from test file.
-- Use in cfx_cttgsubtable.cpp

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