tree 47e9559c28145c097d6be866e4d54f950f144b49
parent d928460c8d68bc4455b4a0664899a846e8327c49
author Ben Hoyt <benhoyt@gmail.com> 1661404594 +1200
committer GitHub <noreply@github.com> 1661404594 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJjBwWyCRBK7hj4Ov3rIwAAyfMIAEZQvGQTp3DU8RXAw8IY0N+I
 VBCgeGXqOBKz1JDoHsTEYe+2mOFGta5qxXOUkqmjj96GEC377k5qud3EVaG1OqoO
 0hwOp2fyJaCiqhX+4T00ZrUf8Ze33TYQOmFy8nPyjP69+YtvXqFat2sfjYtvL32H
 iPFa9k10WJAG5Q2j1hvWoZbG5guURZM/WKBo3EN97OfVK0iwn2FFZmGUg0Pu0SIT
 R//Yug4imh5QeUeaXWedWpV9nQifuWJNXi8LVLB4VVMAnBiI44JPF5yKjpTtHvvM
 Cuqy2QNBFBZDH8UaJqEzSmwz9JkRVqVFasNFy8TcVAhSbiLfL7RLK5Nkm7u6uLU=
 =iIXF
 -----END PGP SIGNATURE-----
 

Catch panics when calling GoString like fmt %#v does (#87)

This handles a few cases (similar to how fmt %#v does):

- A GoString method on a value receiver, called with a nil pointer
- A GoString method on a pointer receiver that doesn't check for nil
- A GoString method that panics in some other way

Because Go 1.17 added a method Time.GoString with value receiver, this
broke structs that had *time.Time fields with nil values (which is
common!).

Also added a bunch of tests for these cases.

Fixes #77

Co-authored-by: Jordan Barrett <jordan.barrett@canonical.com>