)]}'
{
  "commit": "81c6041922266cc20cf2507365cfee3335eb83d4",
  "tree": "5ad69316769b342659d02dc4c2e0e1ee5bd8e71b",
  "parents": [
    "d6290c3b07301936a768f6e0a8455c20076eec17"
  ],
  "author": {
    "name": "Paul Wilkins",
    "email": "paulwilkins@google.com",
    "time": "Fri Oct 05 11:16:46 2012 +0100"
  },
  "committer": {
    "name": "Paul Wilkins",
    "email": "paulwilkins@google.com",
    "time": "Fri Oct 05 11:16:46 2012 +0100"
  },
  "message": "Fix SIMD unsafe use of floating point.\n\nThis commit fixes unsafe simd / floating point interactions arising\nfrom the current hybrid and 16x16 transform implementation.\nThese led to a raft of bugs and issues when the project was\nbuilt using VS2008 for Win32 though they did not show up with\nthe unix builds.\n\nGerrit makes a meal out of presenting the fix but all I have actually\ndone is indent the body of each function that uses floating point by\none level and bracket with emms instructions using  the function\nvp8_clear_system_state(). See below.\n\nfunction () {\n  vp8_clear_system_state();\n  {\n  ... function body\n  }\n  vp8_clear_system_state();\n}\n\nThis is almost certainly over the top in terms of number of emms\ninstructions but is a temporary measure pending implementation of\ninteger variants of each function to replace the floating point.\n\nLimited testing suggests that this fixes the problems that arose for\nWin32 VS2008 when the hybrid or 16x16 transforms were enabled.\n\nChange-Id: I7c9a72bd79315246ed880578dec51e2b7c178442\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "b4475c628a89adf520a3c595a0db28e5deb61af3",
      "old_mode": 33188,
      "old_path": "vp8/common/idctllm.c",
      "new_id": "5eb9d0c8a66ab880cbd000813a6e9c81edc4b188",
      "new_mode": 33188,
      "new_path": "vp8/common/idctllm.c"
    },
    {
      "type": "modify",
      "old_id": "209ba20b5809c14e89f21fdd417c239b7b3aedea",
      "old_mode": 33188,
      "old_path": "vp8/encoder/dct.c",
      "new_id": "81b4e6be2454969f92873b8f229d6fe650bc0b55",
      "new_mode": 33188,
      "new_path": "vp8/encoder/dct.c"
    }
  ]
}
