)]}'
{
  "commit": "510c87381d5e8c28abcdfe9ea0dac817c11ab0b3",
  "tree": "001b6d8ae2ae7cf718292f1e1de53a860df26e33",
  "parents": [
    "6941d8eca7182bf134c76e39fa5f1766074dcac9"
  ],
  "author": {
    "name": "jhorvitz",
    "email": "jhorvitz@google.com",
    "time": "Fri May 07 05:41:30 2021 -0700"
  },
  "committer": {
    "name": "Copybara-Service",
    "email": "copybara-worker@google.com",
    "time": "Fri May 07 05:42:38 2021 -0700"
  },
  "message": "Don\u0027t use a constant for an empty `PlatformMappingValue`.\n\n`PlatformMappingValue` contains strong caches, so using a constant can lead to memory leaks. I observed this in a test I\u0027m working on where `BuildOptions` instances I expected to be GC\u0027d were not.\n\nInstead of a constant, just rely on skyframe to reuse the same empty `PlatformMappingValue`, since there\u0027s only one possible `SkyKey` that can produce this value.\n\nPiperOrigin-RevId: 372538481\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "01497bb6e253ef5829ebeba35d7d038cc2503411",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/skyframe/PlatformMappingFunction.java",
      "new_id": "9e8ac0e7ed55ebd9c95e162b4d74f3bf76db4861",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/skyframe/PlatformMappingFunction.java"
    },
    {
      "type": "modify",
      "old_id": "d684d802047a525cdaf7aaf26d109fdfde41a501",
      "old_mode": 33188,
      "old_path": "src/main/java/com/google/devtools/build/lib/skyframe/PlatformMappingValue.java",
      "new_id": "47d63aabd410a24e521af8e37db5a8492d199a04",
      "new_mode": 33188,
      "new_path": "src/main/java/com/google/devtools/build/lib/skyframe/PlatformMappingValue.java"
    }
  ]
}