• John Koleszar's avatar
    cleanup: collect twopass variables · 63cb1a7c
    John Koleszar authored
    This patch collects the twopass specific memebers of VP8_COMP into a
    dedicated struct. This is a first step towards isolating the two pass
    rate control and aids readability by decorating these variables with
    the 'twopass.' namespace. This makes it clear to the reader in what
    contexts the variable will be valid, and is a hint that a section of
    code might be a good candidate to move to firstpass.c in later
    refactoring. There likely will be other rate control modes that need
    their own specific data as well.
    
    This notation is probably overly verbose in firstpass.c, so an
    alternative would be to access this struct through a pointer like
    'rc->' instead of 'cpi->firstpass.' in that file. Feel free to make
    a review comment to that effect if you prefer.
    
    Change-Id: I0ab8254647cb4b493a77c16b5d236d0d4a94ca4d
    63cb1a7c
picklpf.c 15 KB