commit 19084f89d2ced9926df06fafe122d9ea5595df84
author Elad Alon <eladalon@webrtc.org> Fri Nov 09 21:55:10 2018 +0100
committer Commit Bot <commit-bot@chromium.org> Mon Nov 12 14:05:11 2018 +0000
tree 5a51cda104c7005a686a428867b2d9b2ed6e02e6
parent 49c33ce5286db2661f7639e02b3cd7508370fd61

Event logs - encode N channels as N-1

Since the number of channels is always greater than 0, smaller
deltas can be accomplished by encoding a sequence of (1, 2, 1)
as if the sequence were (0, 1, 0). This way, wrap around to the
first value is a delta of 1, rahter than a delta of 3.

For simplicity's sake, though at the cost of consistency, we still
encode the base event's number of channels unshifted. We do so
because there are no bits to be gained by doing it otherwise, and
the value there is more likely to be manually inspected, than are
the deltas, so a simpler scheme has merit.

Bug: webrtc:8111
Change-Id: I2d4def67da85c42802fe13cd0494fdd9f2b38f7a
Reviewed-on: https://webrtc-review.googlesource.com/c/110242
Commit-Queue: Elad Alon <eladalon@webrtc.org>
Reviewed-by: Björn Terelius <terelius@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25601}