Support for embedded struct pointers?

[pnelson]

Out of habit, my embedded structs have always been pointers. Something like:

type WithEmbeddedStructPtr struct {
    Id int64
    *Names
}

Would it be possible to support them? I see readStructColumns explicitly checks if the anonymous field is a struct, and not the possibility of a reflect.Ptr to a struct. I tried to add it quickly myself but then it broke in get and I had no idea what was going on in there.

I'm not sure if this is possible or if it is worth adding. Thoughts?

[coopernurse]

I suspect this could be supported. A first step would be to submit a PR with one or more failing tests that exercise this. I'll merge it in and disable the test so CI still runs, but we can then track potential solutions against this test case.

[shuhaowu]

+1 for this.