tree 0cede79b38a3095efdacc49b69b66f80bef0a02a
parent d6a2da07d5c90e21aaf7a1612314180db73bdb6b
author William A. Kennington III <wak@google.com> 1556095484 -0700
committer William A. Kennington III <wak@google.com> 1556564795 -0700

message/payload: Clean up check / trailing state

We want to be able to trivially re-use payloads for marshalling data
from a buffer into other formats. This change tries to make the meaning
of trailingOk and unpackCheck consistent, since the meanings didn't seem
clear in the previous code. Now, unpackCheck is only used to determine
if unpacking was checked, and trailingOk determines if unpackCheck is
required.

This also fixes lots of spurious warnings being printed for commands
which were checking their output correctly, or were legacy and unable to
check output.

Change-Id: Id7aa9266693b4e3f896027acf6b3e5d757fdf981
Signed-off-by: William A. Kennington III <wak@google.com>