Fix the error handling for CMaps that fail to load
Reduce ASCII checks in makeInlineImage().

makeInlineImage() has a "are the next five chars ASCII?" check which is
run after an "EI" sequence has been found. This check involves the
creation of a new object because peekBytes() calls subarray().

Unfortunately, the check is currently run on whitespace chars even when
an "EI" sequence has not yet been found, i.e. when it's not needed. For
the PDF in #2618, there are over 820,000 such checks.

This change reworks the relevant loop so that the check is only done
once an "EI" sequence has been seen. This reduces the number of checks
to 157,000, and speeds up rendering by somewhere between 2% and 7% (the
measurements are noisy).
Merge pull request #5187 from nnethercote/EI-checking

Reduce ASCII checks in makeInlineImage().
Merge pull request #5186 from Snuffleupagus/fix-CMap-unavailable-error-handling

Fix the error handling for CMaps that fail to load