container/initplace: remove indirect method

This is no longer useful and is highly error-prone.

Signed-off-by: Ophestra <cat@gensokyo.uk>

2 files changed, 2 insertions, 21 deletions

diff --git a/container/initplace.go b/container/initplace.go
index d08cf653..fe4b60a6 100644
--- a/container/initplace.go
+++ b/container/initplace.go
@@ -22,15 +22,6 @@ func (f *Ops) Place(name *check.Absolute, data []byte) *Ops {
 	return f
 }
 
-// PlaceP is like Place but writes the address of [TmpfileOp.Data] to the pointer dataP points to.
-func (f *Ops) PlaceP(name *check.Absolute, dataP **[]byte) *Ops {
-	t := &TmpfileOp{Path: name}
-	*dataP = &t.Data
-
-	*f = append(*f, t)
-	return f
-}
-
 // TmpfileOp places a file on container Path containing Data.
 type TmpfileOp struct {
 	Path *check.Absolute
diff --git a/container/initplace_test.go b/container/initplace_test.go
index a330a804..03f298b6 100644
--- a/container/initplace_test.go
+++ b/container/initplace_test.go
@@ -82,18 +82,8 @@ func TestTmpfileOp(t *testing.T) {
 	})
 
 	checkOpsBuilder(t, []opsBuilderTestCase{
-		{"noref", new(Ops).Place(samplePath, sampleData), Ops{
-			&TmpfileOp{
-				Path: samplePath,
-				Data: sampleData,
-			},
-		}},
-
-		{"ref", new(Ops).PlaceP(samplePath, new(*[]byte)), Ops{
-			&TmpfileOp{
-				Path: samplePath,
-				Data: []byte{},
-			},
+		{"full", new(Ops).Place(samplePath, sampleData), Ops{
+			&TmpfileOp{Path: samplePath, Data: sampleData},
 		}},
 	})