tree 071af2be9284a58acaf866dc2df590b2582572ad
parent 9b27f70bdee85b130d92ea2531563d6c132b8d9b
author Matthew Maurer <mmaurer@google.com> 1554855715 -0700
committer Matthew Maurer <mmaurer@google.com> 1555969968 +0000

Fix C literal usage

There was a lot of usage of implicit sign extension/casting to represent
max types, most comonly `~0` being implicitly cast to `uint64_t`,
resulting in a sign extension followed by reinterpreting the sign bit.
This is fragile, as `~0U` will only set the *low* 32 bits of `uint64_t`
despite appearing more correct.

This patch removes the use of implicit casting, and uses _MAX
or _INVALID macros where reasonable rather than ~0.

Bug: 129300035
Change-Id: Idb86b05d2fc4c776e9be0ebe423f31e33841c9a4