tailcfg: remove outdated comments about Clone methods

The cloner tool adds static checks that the Clone methods are up to
date, so failing to update Clone causes a compiler error.

Signed-off-by: David Crawshaw <crawshaw@tailscale.com>
josh/no-goroutine-per-udp-read-2
David Crawshaw 4 years ago committed by David Crawshaw
parent 0710fca0cd
commit 2c48b4ee14

@ -112,8 +112,6 @@ type User struct {
Logins []LoginID Logins []LoginID
Roles []RoleID Roles []RoleID
Created time.Time Created time.Time
// Note: be sure to update Clone when adding new fields
} }
type Login struct { type Login struct {
@ -156,9 +154,6 @@ type Node struct {
KeepAlive bool // open and keep open a connection to this peer KeepAlive bool // open and keep open a connection to this peer
MachineAuthorized bool // TODO(crawshaw): replace with MachineStatus MachineAuthorized bool // TODO(crawshaw): replace with MachineStatus
// NOTE: any new fields containing pointers in this type
// require changes to Node.Clone.
} }
type MachineStatus int type MachineStatus int
@ -275,9 +270,6 @@ type Service struct {
Description string `json:",omitempty"` // text description of service Description string `json:",omitempty"` // text description of service
// TODO(apenwarr): allow advertising services on subnet IPs? // TODO(apenwarr): allow advertising services on subnet IPs?
// TODO(apenwarr): add "tags" here for each service? // TODO(apenwarr): add "tags" here for each service?
// NOTE: any new fields containing pointers in this type
// require changes to Hostinfo.Clone.
} }
// Hostinfo contains a summary of a Tailscale host. // Hostinfo contains a summary of a Tailscale host.
@ -302,7 +294,7 @@ type Hostinfo struct {
NetInfo *NetInfo `json:",omitempty"` NetInfo *NetInfo `json:",omitempty"`
// NOTE: any new fields containing pointers in this type // NOTE: any new fields containing pointers in this type
// require changes to Hostinfo.Clone and Hostinfo.Equal. // require changes to Hostinfo.Equal.
} }
// NetInfo contains information about the host's network state. // NetInfo contains information about the host's network state.
@ -354,7 +346,7 @@ type NetInfo struct {
// the control plane. // the control plane.
DERPLatency map[string]float64 `json:",omitempty"` DERPLatency map[string]float64 `json:",omitempty"`
// Update Clone and BasicallyEqual when adding fields. // Update BasicallyEqual when adding fields.
} }
func (ni *NetInfo) String() string { func (ni *NetInfo) String() string {

Loading…
Cancel
Save