document requirements for merge function

gentele · gentele · commit 5a0fa359393d · 2018-10-25T19:12:25.000+02:00
diff --git a/pkg/devspace/config/configutil/merge.go b/pkg/devspace/config/configutil/merge.go
@@ -13,6 +13,8 @@ func Merge(object interface{}, overwriteObject interface{}) {
 	merge(&object, overwriteObject, objectPointerUnsafe, overwriteObjectPointerUnsafe)
 }
 
+// objectPointer MUST be a pointer of a pointer
+// objectPointerUnsafe MUST be an unsafe pointer of a pointer
 func merge(objectPointer interface{}, overwriteObjectPointer interface{}, objectPointerUnsafe unsafe.Pointer, overwriteObjectPointerUnsafe unsafe.Pointer) {
 	overwriteObjectRef := reflect.ValueOf(overwriteObjectPointer)
 

Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,8 @@ func Merge(object interface{}, overwriteObject interface{}) {`
`13`	`13`	`merge(&object, overwriteObject, objectPointerUnsafe, overwriteObjectPointerUnsafe)`
`14`	`14`	`}`
`15`	`15`
	`16`	`+// objectPointer MUST be a pointer of a pointer`
	`17`	`+// objectPointerUnsafe MUST be an unsafe pointer of a pointer`
`16`	`18`	`func merge(objectPointer interface{}, overwriteObjectPointer interface{}, objectPointerUnsafe unsafe.Pointer, overwriteObjectPointerUnsafe unsafe.Pointer) {`
`17`	`19`	`overwriteObjectRef := reflect.ValueOf(overwriteObjectPointer)`
`18`	`20`