| | unit.SetAddress now takes a vararg Update LP#1298770 The previous branch updated instance.NewAddress to be varargs and demonstated that most callers of NewAddresses were used to get a slice of addresses so they could pass them to unit.SetAddresses. By making SetAddresses vararg NewAddresses is unneeded. This will be next change. https://code.launchpad.net/~dave-cheney/juju-core/108-set-addresses-take-varargs/+merge/213393 (do not edit description out of merge proposal) | Unified diffs | Side-by-side diffs | Delta from patch set | Stats (+45 lines, -65 lines) | Patch |  | A | [revision details] | View | 1 2 3 4 | 1 chunk | +2 lines, -0 lines | 0 comments | Download | | M | cmd/juju/scp_test.go | View | | 1 chunk | +1 line, -1 line | 0 comments | Download | | M | cmd/juju/ssh_test.go | View | | 1 chunk | +1 line, -1 line | 0 comments | Download | | M | cmd/juju/status_test.go | View | 1 2 3 | 1 chunk | +1 line, -1 line | 0 comments | Download | | M | cmd/jujud/machine_test.go | View | 1 2 | 1 chunk | +1 line, -1 line | 0 comments | Download | | M | juju/testing/utils.go | View | | 1 chunk | +1 line, -3 lines | 0 comments | Download | | M | state/api/deployer/deployer_test.go | View | | 2 chunks | +2 lines, -5 lines | 0 comments | Download | | M | state/api/machiner/machiner_test.go | View | | 1 chunk | +1 line, -1 line | 0 comments | Download | | M | state/api/provisioner/provisioner_test.go | View | 1 2 | 2 chunks | +2 lines, -4 lines | 0 comments | Download | | M | state/apiserver/client/client_test.go | View | | 4 chunks | +6 lines, -6 lines | 0 comments | Download | | M | state/apiserver/client/run_test.go | View | | 2 chunks | +2 lines, -2 lines | 0 comments | Download | | M | state/apiserver/deployer/deployer_test.go | View | | 2 chunks | +2 lines, -6 lines | 0 comments | Download | | M | state/apiserver/uniter/uniter_test.go | View | 1 2 | 1 chunk | +1 line, -3 lines | 0 comments | Download | | M | state/machine.go | View | | 1 chunk | +1 line, -1 line | 0 comments | Download | | M | state/machine_test.go | View | 1 2 3 | 1 chunk | +1 line, -1 line | 0 comments | Download | | M | state/megawatcher_internal_test.go | View | | 1 chunk | +1 line, -2 lines | 0 comments | Download | | M | state/state_test.go | View | | 1 chunk | +5 lines, -5 lines | 0 comments | Download | | M | state/unit_test.go | View | 1 2 3 | 3 chunks | +5 lines, -7 lines | 0 comments | Download | | M | worker/instancepoller/machine_test.go | View | | 1 chunk | +1 line, -1 line | 0 comments | Download | | M | worker/instancepoller/updater.go | View | | 2 chunks | +2 lines, -2 lines | 0 comments | Download | | M | worker/provisioner/kvm-broker_test.go | View | 1 2 | 1 chunk | +1 line, -3 lines | 0 comments | Download | | M | worker/provisioner/lxc-broker_test.go | View | 1 2 | 1 chunk | +1 line, -3 lines | 0 comments | Download | | M | worker/provisioner/provisioner_test.go | View | 1 2 | 1 chunk | +1 line, -3 lines | 0 comments | Download | | M | worker/uniter/uniter_test.go | View | 1 2 3 4 | 1 chunk | +3 lines, -3 lines | 0 comments | Download | Total messages: 4 |
