)]}'
{
  "commit": "43e6281f629256ba5ad1d51c845294aea7e36d6a",
  "tree": "b19f15f6b9d0fc5b0661df1efeed476971a8140d",
  "parents": [
    "c58d95717fb1a3ccab3bcad94731a02ecd1af3e7"
  ],
  "author": {
    "name": "Urvang Joshi",
    "email": "urvang@google.com",
    "time": "Thu Oct 20 14:51:01 2016 -0700"
  },
  "committer": {
    "name": "Urvang Joshi",
    "email": "urvang@google.com",
    "time": "Thu Oct 20 14:51:01 2016 -0700"
  },
  "message": "Encoder/Decoder mismatch fix: need a separate copy of eob_counts.\n\nThe bug was introduced here:\nhttps://chromium-review.googlesource.com/#/c/399975/4/av1/encoder/bitstream.c\nIn that patch, I had removed 2nd declaration of a variable of the same\nname. But it turns out that the two variables actually had a different\ntype (even though the name was same).\n\nNow, we keep both variables, but rename one of them -- that fixes the\nmismatch. While we are at it, made both variables local as well.\n\nThe fix can be verified as follows:\n../../libvpx/configure --enable-experimental --enable-supertx\n--enable-var-tx --enable-entropy --enable-internal-stats \u0026\u0026 make clean\n\u0026\u0026 make -j16\n\naomenc -o soccer_cif_1000_av1_b8.webm ../soccer_cif.y4m --codec\u003dav1\n--limit\u003d50 --skip\u003d0 -p 2 --pass\u003d1 --fpf\u003dsoccer_cif_av1.fpf --good\n--cpu-used\u003d0 --target-bitrate\u003d1000 --lag-in-frames\u003d25 --min-q\u003d0\n--max-q\u003d63 --auto-alt-ref\u003d1 --kf-max-dist\u003d150 --kf-min-dist\u003d0\n--drop-frame\u003d0 --static-thresh\u003d0 --bias-pct\u003d50 --minsection-pct\u003d0\n--maxsection-pct\u003d2000 --arnr-maxframes\u003d7 --arnr-strength\u003d5 --sharpness\u003d0\n--undershoot-pct\u003d100 --overshoot-pct\u003d100 --frame-parallel\u003d0\n--tile-columns\u003d0 --profile\u003d0 --test-decode\u003dwarn\n\naomenc -o soccer_cif_1000_av1_b8.webm ../soccer_cif.y4m --codec\u003dav1\n--limit\u003d50 --skip\u003d0 -p 2 --pass\u003d2 --fpf\u003dsoccer_cif_av1.fpf --good\n--cpu-used\u003d0 --target-bitrate\u003d1000 --lag-in-frames\u003d25 --min-q\u003d0\n--max-q\u003d63 --auto-alt-ref\u003d1 --kf-max-dist\u003d150 --kf-min-dist\u003d0\n--drop-frame\u003d0 --static-thresh\u003d0 --bias-pct\u003d50 --minsection-pct\u003d0\n--maxsection-pct\u003d2000 --arnr-maxframes\u003d7 --arnr-strength\u003d5 --sharpness\u003d0\n--undershoot-pct\u003d100 --overshoot-pct\u003d100 --frame-parallel\u003d0\n--tile-columns\u003d0 --profile\u003d0 --test-decode\u003dwarn -v --psnr\n\nChange-Id: Ibd72dbe1f620e6de231513220ee4e190606613ae\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "a4bd8d96fd13cbfcaa87106d21c579732e906e56",
      "old_mode": 33188,
      "old_path": "av1/encoder/bitstream.c",
      "new_id": "743d10b89b93abc12cce9df4919cf360e73d87bb",
      "new_mode": 33188,
      "new_path": "av1/encoder/bitstream.c"
    }
  ]
}