[ Quoting <notifications@github.com> in "Re: [dns] Parsing and Stringer Chan..." ]
> > +func unpackTXTString(msg []byte, offset int) (string, int, error) {
> 
> I think unpackTXTString and packTXTString should get called for the types that have single strings like NAPTR. Need to double check that.
> 
> Renaming from TXT to Txt is a good call.

Ok,

I've checked out the pull request to play some more. But the TestTXTRRQuick
fails for me:

i=== RUN TestTXTRRQuick
--- FAIL: TestTXTRRQuick (0.01 seconds)
        parse_test.go:227: Parsed rr.String() doesn't match original string
        parse_test.go:228: Original: @w\229\)\000B1w.   1       IN      TXT     "\137\185\201\0152:\1690\176)r']" "\133\184\020\202\232\198\011\167\155\014l\n\190\172\225\183\030\227&m\209\004=\153\193\204rT\209\238r6b\191\154HB\149" "\249*d\248\227\130\166@y5M\189j\176\220.\134\220\180\030\217L8\179a\168\156\174{\184\131\172\204|\244A\223\008\237(\158C\240\181\171\204Cz\241\250\184 y~Q\152\011\196\127"
        parse_test.go:229:   Parsed: .  1       IN      TXT     "\137\185\201\0152:\1690\176)r']" "\133\184\020\202\232\198\011\167\155\014l\n\190\172\225\183\030\227&m\209\004=\153\193\204rT\209\238r6b\191\154HB\149" "\249*d\248\227\130\166@y5M\189j\176\220.\134\220\180\030\217L8\179a\168\156\174{\184\131\172\204|\244A\223\008\237(\158C\240\181\171\204Cz\241\250\184 y~Q\152\011\196\127"
        parse_test.go:255: #26: failed on input 1780756845054258976
FAIL

I also think the types that have Txt members can benefit from a little
documentation on how the Txt []string behave, for other records we can just
refer back to the TXT record.

/Miek

-- 
Miek Gieben