Skip to content
Snippets Groups Projects
  1. Feb 19, 2025
  2. Feb 18, 2025
  3. Feb 13, 2025
  4. Feb 12, 2025
  5. Feb 10, 2025
    • Timothy B. Terriberry's avatar
      Remove commented-out code. · 734aed05
      Timothy B. Terriberry authored
      
      Git will now remember it for us forever.
      
      Signed-off-by: default avatarJean-Marc Valin <jeanmarcv@google.com>
      734aed05
    • Timothy B. Terriberry's avatar
      Always use L=0 for last repeated long extension. · 60d6c560
      Timothy B. Terriberry authored
      
      Even if it is followed by repeated short extensions with payloads.
      We track the total size of the short extension payloads that need
       to be repeated, and remove that from the available space for the
       last long extension.
      This means we can no longer use L=0 on a repeat to skip coding a
       frame separator when the extensions to be repeated contain a long
       extension followed by one or more short extensions with payloads
       (and there are no more non-repeated extensions in the current
       frame, but there are extensions in the next frame), but this case
       seems uncommon, and hard to explain.
      The savings from always being able to skip coding a length when the
       final extensions are repeated extensions with at least one long
       extension is likely higher.
      We can still skip a frame separator if we repeat only short
       extensions.
      
      Also update existing tests and add a test for the case where we do
       not have enough space for the repeated short extensions after the
       last long extension.
      
      Signed-off-by: default avatarJean-Marc Valin <jeanmarcv@google.com>
      60d6c560
    • Timothy B. Terriberry's avatar
      Advance frame number after an L=0 RTE. · b49c07d1
      Timothy B. Terriberry authored
      
      This avoids the need to code a frame separator in the case that the
       last repeated extension is a short extension, and the next
       non-repeated extension is in the following frame instead of the
       current one (saving one additional byte).
      Also add tests for both encoding and decoding this.
      
      Signed-off-by: default avatarJean-Marc Valin <jeanmarcv@google.com>
      b49c07d1
Loading