1. 29 Oct, 2010 1 commit
  2. 28 Oct, 2010 3 commits
    • Timothy B. Terriberry's avatar
      Eliminate more warnings. · 97b766a4
      Timothy B. Terriberry authored
      This eliminates a large set of warnings exposed by the Mozilla build
       system (Use of C++ comments in ISO C90 source, commas at the end of
       enum lists, a couple incomplete initializers, and signed/unsigned
       comparisons).
      It also eliminates many (but not all) of the warnings expose by newer
       GCC versions and _FORTIFY_SOURCE (e.g., calling fread and fwrite
       without checking the return values).
      There are a few spurious warnings left on my system:
      
      ../vp8/encoder/encodemb.c:274:9: warning: 'sz' may be used
       uninitialized in this function
      gcc seems to be unable to figure out that the value shortcut doesn't
       change between the two if blocks that test it here.
      
      ../vp8/encoder/onyx_if.c:5314:5: warning: comparison of unsigned
       expression >= 0 is always true
      ../vp8/encoder/onyx_if.c:5319:5: warning: comparison of unsigned
       expression >= 0 is always true
      This is true, so far as it goes, but it's comparing against an enum,
       and the C standard does not mandate that enums be unsigned, so the
       checks can't be removed.
      
      Change-Id: Iead6cd561a2afaa3d801fd63f1d8d58953da7426
      97b766a4
    • Frank Galligan's avatar
      Output the PSNR for the entire file. · 483ce403
      Frank Galligan authored
      If --psnr option is enabled vpxenc will output PSNR values for the
      entire file. Added a \n before final output to make sure the output
      is on its own line. Overall and Avg psnr matches the values written
      to opsnr.stt file.
      
      Change-Id: Ibac5fa9baf8d5a626ea0d6ba161b484e6e8427ee
      483ce403
    • Timothy B. Terriberry's avatar
      Eliminate more warnings. · c4d7e5e6
      Timothy B. Terriberry authored
      This eliminates a large set of warnings exposed by the Mozilla build
       system (Use of C++ comments in ISO C90 source, commas at the end of
       enum lists, a couple incomplete initializers, and signed/unsigned
       comparisons).
      It also eliminates many (but not all) of the warnings expose by newer
       GCC versions and _FORTIFY_SOURCE (e.g., calling fread and fwrite
       without checking the return values).
      There are a few spurious warnings left on my system:
      
      ../vp8/encoder/encodemb.c:274:9: warning: 'sz' may be used
       uninitialized in this function
      gcc seems to be unable to figure out that the value shortcut doesn't
       change between the two if blocks that test it here.
      
      ../vp8/encoder/onyx_if.c:5314:5: warning: comparison of unsigned
       expression >= 0 is always true
      ../vp8/encoder/onyx_if.c:5319:5: warning: comparison of unsigned
       expression >= 0 is always true
      This is true, so far as it goes, but it's comparing against an enum, and the C
       standard does not mandate that enums be unsigned, so the checks can't be
       removed.
      
      Change-Id: Iaf689ae3e3d0ddc5ade00faa474debe73b8d3395
      c4d7e5e6
  3. 27 Oct, 2010 2 commits
    • Frank Galligan's avatar
      Output the PSNR for the entire file. · 3d84da6b
      Frank Galligan authored
      If --psnr option is enabled vpxenc will output PSNR values for the
      entire file. Added a \n before final output to make sure the output
      is on its own line. Overall and Avg psnr matches the values written
      to opsnr.stt file.
      
      Change-Id: I869268b704fe8b0c8389d318cceb6072fea102f8
      3d84da6b
    • John Koleszar's avatar
      vpxenc: add unique track id · c7c0c056
      John Koleszar authored
      MKV requires a unique(ish) TrackID element in the track info header.
      Instead of the current hard-coded ID, take a hash of the video track
      and use that. This value is not written in the deterministic output
      mode, despite being a deterministic value itself, to give flexibility
      to change the hash algorithm and not affect bisecting across the
      change.
      
      Change-Id: I807fc3ea6d1427a151c3ef703269b67e80aef860
      c7c0c056
  4. 26 Oct, 2010 8 commits
    • John Koleszar's avatar
      vpxenc: add deterministic output option · 75afcee9
      John Koleszar authored
      By baking the version number into the output file, a hash of the file
      will vary from commit to commit, even if the output is otherwise bit
      exact. Add a -D option to suppress this behavior, for use when
      bisecting or other debugging.
      
      Change-Id: I5089a8ce5719920ffaf47620fa9069b81fa15673
      75afcee9
    • John Koleszar's avatar
      vpxenc: warn against webm output to pipes · 0a6bf29e
      John Koleszar authored
      The WebM writer requires a seekable stream.
      
      Change-Id: I192e00706a0685362d41b8d2faf80add63d564b9
      0a6bf29e
    • John Koleszar's avatar
      vpxenc: specify output file with -o · 5329189a
      John Koleszar authored
      Requiring the output file to be specified with the -o option opens up
      the possibility of supporting multiple input files in the future.
      
      Change-Id: I14c9b75e9b21184b47081e1ccf30cf4c91315964
      5329189a
    • John Koleszar's avatar
      vpxenc: change --framerate to --fps · 28f177cd
      John Koleszar authored
      Saves a little typing. FPS is a well known abbreviation.
      
      Change-Id: I53730ea36afb9309732eb1c72c52d824d5365fec
      28f177cd
    • John Koleszar's avatar
      vpxenc: output webm by default · 456bfb19
      John Koleszar authored
      WebM should be preferred to IVF output, since it has wider tool support.
      
      Change-Id: I5ac3d5cb68722e6c8af917cdba32ac01dd5e0ea2
      456bfb19
    • John Koleszar's avatar
      rename ivf{enc,dec} to vpx{enc,dec} · ea68ee04
      John Koleszar authored
      The new WebM output support should be preferred to IVF, but we can't
      change the default behavior of the ivf* tools. There are a few other
      default behaviors for these tools that are counterintuitive for
      historical reasons, and changing the binary name provides the
      opportunity to clean those up as well. This patch takes the first
      step by renaming the binaries.
      
      Change-Id: I647008ae37cc352dd27ec1da7ed13489e0609b24
      ea68ee04
    • John Koleszar's avatar
      ivfenc: webm output support · dc66630c
      John Koleszar authored
      This patch adds the --webm option, to allow the creation of WebM streams
      without having to remux ivf into webm.
      
      Change-Id: Ief93c114a6913c55a04cf51bce38f594372d0ad0
      dc66630c
    • Frank Galligan's avatar
      Fixed the timebase parameter of ivfenc. · 1258cf62
      Frank Galligan authored
      Ivfenc will use timebase if it is set. If it is not set ivfenc will
      still double the timebase so altref frames will have a unique pts.
      Patch Set #3: Use integer math to generate source pts. Added a
      framerate parameter. Increased the default timebase to milliseconds to
      remove the *2 everywhere.
      
      Change-Id: I8d25b5b2cb26deef7eb72d74b5f76c98cafaf4db
      1258cf62
  5. 11 Oct, 2010 1 commit
  6. 09 Sep, 2010 1 commit
  7. 27 Aug, 2010 1 commit
    • Timothy B. Terriberry's avatar
      Fix two-pass framrate for Y4M input. · e105e245
      Timothy B. Terriberry authored
      The timebase was being set to the value in the Y4M file on each
       pass, but only doubled to account for the altref placement on
       the first past.
      This avoids reseting it on the second pass.
      
      Change-Id: Ie342639bad1ffe9c2214fbbaaded72cfed835b42
      e105e245
  8. 24 Jun, 2010 1 commit
    • John Koleszar's avatar
      ivfenc: correct fixed kf interval, --disable-kf · b1e36f28
      John Koleszar authored
      ivfenc was setting the VPX_KF_FIXED mode when the kf_min_dist and
      kf_max_dist parameters were set to each other. This flag actually means
      that keyframes are disabled, and that name was deprecated to avoid
      confusion such as this. Instead, a new option is exposed for setting the
      VPX_KF_DISABLED mode, and the intervals are passed through to the codec,
      which will do automatic placement at a fixed interval as expected.
      
      Change-Id: I15abbec5936f39d5901878b4bc154372fbc23a43
      b1e36f28
  9. 16 Jun, 2010 1 commit
    • Tom Finegan's avatar
      Avoid encoding garbage when ivfenc encounters an unsupported Y4M file. · e703af97
      Tom Finegan authored
      This change stops ivfenc from treating unsupported Y4M files as raw
      input.
      
      For example, if given an interlaced Y4M file, ivfenc treated the input
      as if it were raw data because the unsupported Y4M file case previously
      fell through without being handled.
      
      Change-Id: I06caa50f3448e6388741a77346daaebf77c277e1
      e703af97
  10. 14 Jun, 2010 1 commit
    • John Koleszar's avatar
      ivfenc: fix two-pass support of raw files · df2c62d3
      John Koleszar authored
      Commit 3245d463 "ivfenc: support reading/writing from a pipe" broke
      support for two pass encodes of raw files when done in two
      invocations of ivfenc. The raw image was only set up on pass 0,
      which was never hit when running with --pass=2 --passes=2.
      
      Change-Id: I6a9858be1a8998d5bd45331123b46b1baa05b379
      df2c62d3
  11. 11 Jun, 2010 2 commits
  12. 04 Jun, 2010 1 commit
  13. 03 Jun, 2010 1 commit
  14. 28 May, 2010 1 commit
  15. 25 May, 2010 2 commits
    • John Koleszar's avatar
      vpx_image: add VPX_ prefix to PLANE_* · b6c71918
      John Koleszar authored
      The PLANE_{PACKED,Y,U,V,ALPHA} macros should be renamed to be within the
      VPX_ namespace.
      
      Fixes #27
      b6c71918
    • John Koleszar's avatar
      install includes in DIST_DIR/include/vpx, move vpx_codec/ to vpx/ · b7492341
      John Koleszar authored
      This renames the vpx_codec/ directory to vpx/, to allow applications
      to more consistently reference these includes with the vpx/ prefix.
      This allows the includes to be installed in /usr/local/include/vpx
      rather than polluting the system includes directory with an
      excessive number of includes.
      
      Change-Id: I7b0652a20543d93f38f421c60b0bbccde4d61b4f
      b7492341
  16. 21 May, 2010 2 commits
    • James Zern's avatar
      Put img_fmt in the vpx namespace · 6cd4a10e
      James Zern authored
      Avoid an potential name clashes and match other external types.
      s/IMG_FMT/VPX_$&/g
      s/img_fmt/vpx_$&/g
      
      Change-Id: Ia7ad5bbb6424416b37e71e5f5eb1eca31c3c707f
      6cd4a10e
    • John Koleszar's avatar
      configure: remove HAVE_CONFIG_H · 1df0314e
      John Koleszar authored
      This doesn't play well with autotools, and the preprocessor magic is
      confusing and unhelpful in the vp8-only context.
      
      Change-Id: I2fcb57e6eb7876ecb58509da608dc21f26077ff1
      1df0314e
  17. 18 May, 2010 1 commit