)]}'
{
  "commit": "053fdbcef55e8f977d8decc0fde2920c61eb5374",
  "tree": "9cfbd89321a75cf5de6889cef57c482cc44dc7b6",
  "parents": [
    "bf84681c4a0185014c089cffd533e22bbeffcb49"
  ],
  "author": {
    "name": "Alex Putman",
    "email": "aputman@google.com",
    "time": "Tue May 05 15:26:25 2026 -0400"
  },
  "committer": {
    "name": "Gopher Robot",
    "email": "gobot@golang.org",
    "time": "Wed May 06 09:41:42 2026 -0700"
  },
  "message": "go/analysis/passes/modernize: minmax: fix pure operands only\n\nIf the condition inside the \"if\" statement contains an expression that causes side effects, we don\u0027t want to rewrite with \"min\" or \"max\". The fix might otherwise change the cardinality of effects.\n\nFixes golang/go#77558\n\nChange-Id: I37ba9b6d6c9c014a375328da0e721d0ea6569564\nReviewed-on: https://go-review.googlesource.com/c/tools/+/774401\nReviewed-by: Alan Donovan \u003cadonovan@google.com\u003e\nAuto-Submit: Alex Putman \u003caputman@google.com\u003e\nLUCI-TryBot-Result: golang-scoped@luci-project-accounts.iam.gserviceaccount.com \u003cgolang-scoped@luci-project-accounts.iam.gserviceaccount.com\u003e\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "93aadf04a13fb57fb934a2d701fadf439a33524c",
      "old_mode": 33188,
      "old_path": "go/analysis/passes/modernize/minmax.go",
      "new_id": "08dee962793707bec25172787f968951dfe9318f",
      "new_mode": 33188,
      "new_path": "go/analysis/passes/modernize/minmax.go"
    },
    {
      "type": "modify",
      "old_id": "b1a47998bf6d39a62c5f571407a64858ad4cef9c",
      "old_mode": 33188,
      "old_path": "go/analysis/passes/modernize/modernize_test.go",
      "new_id": "9532c0c62b92dd7e788ed2d12fa046211aae5b96",
      "new_mode": 33188,
      "new_path": "go/analysis/passes/modernize/modernize_test.go"
    },
    {
      "type": "add",
      "old_id": "0000000000000000000000000000000000000000",
      "old_mode": 0,
      "old_path": "/dev/null",
      "new_id": "0c25b10587bde1fcc13711482bf7598234df11f9",
      "new_mode": 33188,
      "new_path": "go/analysis/passes/modernize/testdata/src/minmax/parametersideeffect/parametersideeffect.go"
    },
    {
      "type": "add",
      "old_id": "0000000000000000000000000000000000000000",
      "old_mode": 0,
      "old_path": "/dev/null",
      "new_id": "b0128bfd1e4fe6ad69e1e20271c72eb92100dbf6",
      "new_mode": 33188,
      "new_path": "go/analysis/passes/modernize/testdata/src/minmax/parametersideeffect/parametersideeffect.go.golden"
    }
  ]
}
