remove DefaultMachineGuest(), made redundant after a rebase

Author: Allison Pierson

api/machine_types.go

@@ -407,12 +407,6 @@ var MachinePresets map[string]*MachineGuest = map[string]*MachineGuest{
 	"performance-16x": {CPUKind: "performance", CPUs: 16, MemoryMB: 16 * MIN_MEMORY_MB_PER_CPU},
 }
 
-func DefaultMachineGuest() *MachineGuest {
-	// This assumes that MachineGuest does not contain any pointers.
-	g := *MachinePresets[DefaultVMSize]
-	return &g
-}
-
 type MachineMetrics struct {
 	Port int    `toml:"port" json:"port,omitempty"`
 	Path string `toml:"path" json:"path,omitempty"`

api/machine_types_test.go

@@ -159,19 +159,6 @@ func TestMachineGuest_ToSize(t *testing.T) {
 	}
 }
 
-func TestMachineGuest_DefaultCopied(t *testing.T) {
-
-	m1 := DefaultMachineGuest()
-	m2 := DefaultMachineGuest()
-
-	m1.CPUs = 100
-	m2.CPUs = 200
-
-	if m1.CPUs == m2.CPUs {
-		t.Errorf("DefaultMachineGuest is not copying the guest, it's returning a direct pointer to the default.")
-	}
-}
-
 func TestMachineMostRecentStartTimeAfterLaunch(t *testing.T) {
 	type testcase struct {
 		name        string