tree 89dae5644a3db2c85a5579f3cfe3a11f99d3eafe
parent 2d7eb8bce5c413e0944917cdfd378c26f4e3f85b
author David Turner <david.turner@argondesign.com> 1544196336 +0000
committer David Turner <david.turner@argondesign.com> 1544196561 +0000

Cleanup, remove side-effects from bitstream

write_sequence_header() in bitstream.c was initialising/modifying lots
of variables which did not seem to be its responsibility.  I move a lot
of these assignments to init_seq_coding_tools which initialises
seq_params: I am generally trying to remove side-effects from under
av1_pack_bitstream and move assignments/initialisations to more
appropriate locations.

Various other clean-up:

* Remove type-punning being used to dodge a const
* Use locals for mi pointers where possible
* Rename wiener_info and sgrproj_info to a more informative
  ref_wiener_info and ref_sgrproj_info
* Remove a long-outdated comment
* Remove some now unused variables and make more things const.

This forms part of wider restructuring and refactoring in order to
achieve a clean API separation at the entry to the low-level encoder.

BUG=aomedia:2244

Change-Id: I74f45a9a94a8126420c9ddc656c31300d7288ce0