)]}'
{
  "commit": "15ad38f4fde9cd8366afd8e538df4f73b53696e2",
  "tree": "4896458f242a78ed4c1ec7977af003f64305830f",
  "parents": [
    "4137f7daaa1c3947bba45681a4ff32142ca3097d"
  ],
  "author": {
    "name": "Wan-Teh Chang",
    "email": "wtc@google.com",
    "time": "Thu Aug 16 10:31:53 2018 -0700"
  },
  "committer": {
    "name": "Wan-Teh Chang",
    "email": "wtc@google.com",
    "time": "Fri Sep 07 18:17:17 2018 +0000"
  },
  "message": "Pass n to get_msb in the primitive_quniform funcs.\n\nThe functions this CL modifies were added in the following two CLs:\nhttps://aomedia-review.googlesource.com/c/aom/+/8789\nhttps://aomedia-review.googlesource.com/c/aom/+/20820\n\nThese functions pass n - 1 to get_msb(). I believe these functions\nimplement the ns(n) and NS(n) descriptors of the AV1 spec. ns(n) and\nNS(n) pass n, not n - 1, to FloorLog2(). (FloorLog2() is the same as\nlibaom\u0027s get_msb() function.)\n\nA careful analysis of the code shows passing n instead of n - 1 to\nget_msb() results in the same outcome for these functions. Passing n\nmatches the AV1 spec and is slightly more efficient when n is a power\nof 2.\n\nChange-Id: Ibeff837059e9c9de2b4ab9cb857a2ea60b1c4767\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "23e612102b1bd29debf53dd77d9b9eb08bc2a6c2",
      "old_mode": 33188,
      "old_path": "aom_dsp/binary_codes_reader.c",
      "new_id": "01088010a1f9ece5709d1323ac48d7c720ef20ba",
      "new_mode": 33188,
      "new_path": "aom_dsp/binary_codes_reader.c"
    },
    {
      "type": "modify",
      "old_id": "4e53b156ca80e9babbaebaad13772648ba496d23",
      "old_mode": 33188,
      "old_path": "aom_dsp/binary_codes_writer.c",
      "new_id": "ee7a9f567e7f49fde772c82352c5efb03c9e3397",
      "new_mode": 33188,
      "new_path": "aom_dsp/binary_codes_writer.c"
    }
  ]
}