search

varlink / go

Golang implementation of the Varlink protocol
https://godoc.org/github.com/varlink/go/varlink
Apache License 2.0
55 stars 57 forks source link

Tidy up private reference to public #4

Closed baude closed 5 years ago

baude commented 5 years ago

Signed-off-by: baude bbaude@redhat.com

purpleidea commented 5 years ago

Are you sure this is what you want? I don't know the code base, but this AFAICT changes the semantics substantially, since you're now pointing at a different part of the struct. I didn't look into the code base to see if it matters here, but if anything, the commit message makes no sense.

baude commented 5 years ago

the package wont compile without the change ... so I assume this is what they want. However, it could be the other way around as well.