Merge pull request #32049 from smarterclayton/incorrect_pointer

Automatic merge from submit-queue

generated DeepCopy without a function on a struct pointer is wrong

in and out are both pointers to pointers, so *in has to be cast to
*Type, not Type.

@sttts hit this while debugging something else (most everything should be generated, but in this case the type is wrong)
This commit is contained in:
Kubernetes Submit Queue 2016-09-06 03:14:03 -07:00 committed by GitHub
commit c79a07f21f

View File

@ -609,7 +609,7 @@ func (g *genDeepCopy) doPointer(t *types.Type, sw *generator.SnippetWriter) {
sw.Do("if newVal, err := c.DeepCopy(*in); err != nil {\n", nil)
sw.Do("return err\n", nil)
sw.Do("} else {\n", nil)
sw.Do("*out = newVal.($.|raw$)\n", t.Elem)
sw.Do("*out = newVal.(*$.|raw$)\n", t.Elem)
sw.Do("}\n", nil)
}
}