)]}'
{
  "commit": "1c520054c7a0c55ef2051e8976530e0235cadc35",
  "tree": "ccf7e0832c25200d97a0e539eaf8a1460546c005",
  "parents": [
    "a4bed750abd6904d54f416b22d2c1aa5337332f6"
  ],
  "author": {
    "name": "Wan-Teh Chang",
    "email": "wtc@google.com",
    "time": "Thu Jun 11 10:17:56 2020 -0700"
  },
  "committer": {
    "name": "Wan-Teh Chang",
    "email": "wtc@google.com",
    "time": "Mon Jun 15 18:25:52 2020 +0000"
  },
  "message": "Comments/code cleanup related to aom_codec_encode\n\nImprove the comments and code related to aom_codec_encode when it is\ncalled with the img parameter equal to NULL.\n\nDocument the output parameters time_stamp and time_end and return\nvalues of av1_encode_strategy() and av1_get_compressed_data(). These two\nfunctions are of interest because in the original code, values derived\nfrom the pts and duration parameters of aom_codec_encode() are passed to\nthese two functions in the time_stamp and time_end parameters. By\nshowing that time_stamp and time_end are purely output parameters, we\nconcluded that these two functions ignore the pts and duration\nparameters of aom_codec_encode().\n\nIn encoder_encode() (caller of av1_get_compressed_data() and called by\naom_codec_encode(), the original code uses the local variables\ndst_time_stamp and dst_end_time_stamp for multiple purposes, which makes\nit hard to reason about these two local variables. Replace these two\nlocal variables with single-purpose local variables (and let the\ncompiler figure out how to reuse local variables).\n\nBUG\u003daomedia:2721\n\nChange-Id: Ice8ae7c887c94a103b989bd6fc533fe2af2185ab\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "0ec48c98e3410710db7d229d21f0ecd516a0b5e3",
      "old_mode": 33188,
      "old_path": "aom/aom_encoder.h",
      "new_id": "eddccd998e383e39980830f754e10543d4f73772",
      "new_mode": 33188,
      "new_path": "aom/aom_encoder.h"
    },
    {
      "type": "modify",
      "old_id": "39edd0ff423f923ff24e473f5a5c67869f18b76a",
      "old_mode": 33188,
      "old_path": "av1/av1_cx_iface.c",
      "new_id": "84c872537f80e9a158ff850e121bb3616aa054c0",
      "new_mode": 33188,
      "new_path": "av1/av1_cx_iface.c"
    },
    {
      "type": "modify",
      "old_id": "adc2cc0dd06fded8bbba0ee2fe509c1193a10dbf",
      "old_mode": 33188,
      "old_path": "av1/encoder/encode_strategy.h",
      "new_id": "5f6ee76311378515e23a5fe7126a4044ad3a949d",
      "new_mode": 33188,
      "new_path": "av1/encoder/encode_strategy.h"
    },
    {
      "type": "modify",
      "old_id": "b92b811b42659d4a18779ff186295edb66c20112",
      "old_mode": 33188,
      "old_path": "av1/encoder/encoder.c",
      "new_id": "92de11eff31387bfb0fa6db42a9c44e7ce3c8816",
      "new_mode": 33188,
      "new_path": "av1/encoder/encoder.c"
    },
    {
      "type": "modify",
      "old_id": "6968834d45d14a7cc6607f6b22d69a792d14d422",
      "old_mode": 33188,
      "old_path": "av1/encoder/encoder.h",
      "new_id": "407bea143472de23e5de842a53ab192ec401ebff",
      "new_mode": 33188,
      "new_path": "av1/encoder/encoder.h"
    }
  ]
}
