capnp: add Ptr.TextBytes method

Update FooBytes() code generation to use it.  This should decrease
generated code size and correct the NUL byte check.

capnpc-go/templates/structTextField

@@ -11,18 +11,11 @@ func (s {{.Node.Name}}) {{.Field.Name|title}}() (string, error) {
 
 func (s {{.Node.Name}}) {{.Field.Name|title}}Bytes() ([]byte, error) {
 	p, err := s.Struct.Ptr({{.Field.Slot.Offset}})
-	if err != nil {
-		return nil, err
-	}
 	{{with .Default -}}
-	d := p.DataDefault([]byte({{printf "%q" .}} + "\x00"))
+	return p.TextBytesDefault({{printf "%q" .}}), err
 	{{- else -}}
-	d := p.Data()
+	return p.TextBytes(), err
 	{{- end}}
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
 }
 
 func (s {{.Node.Name}}) Set{{.Field.Name|title}}(v string) error {

internal/aircraftlib/aircraft.capnp.go

@@ -246,14 +246,7 @@ func (s PlaneBase) HasName() bool {
 
 func (s PlaneBase) NameBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(0)
-	if err != nil {
-		return nil, err
-	}
-	d := p.Data()
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytes(), err
 }
 
 func (s PlaneBase) SetName(v string) error {
@@ -1142,14 +1135,7 @@ func (s Z) HasText() bool {
 
 func (s Z) TextBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(0)
-	if err != nil {
-		return nil, err
-	}
-	d := p.Data()
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytes(), err
 }
 
 func (s Z) SetText(v string) error {
@@ -1931,14 +1917,7 @@ func (s Counter) HasWords() bool {
 
 func (s Counter) WordsBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(0)
-	if err != nil {
-		return nil, err
-	}
-	d := p.Data()
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytes(), err
 }
 
 func (s Counter) SetWords(v string) error {
@@ -2154,14 +2133,7 @@ func (s Zjob) HasCmd() bool {
 
 func (s Zjob) CmdBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(0)
-	if err != nil {
-		return nil, err
-	}
-	d := p.Data()
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytes(), err
 }
 
 func (s Zjob) SetCmd(v string) error {
@@ -3276,14 +3248,7 @@ func (s HoldsText) HasTxt() bool {
 
 func (s HoldsText) TxtBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(0)
-	if err != nil {
-		return nil, err
-	}
-	d := p.Data()
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytes(), err
 }
 
 func (s HoldsText) SetTxt(v string) error {
@@ -3915,14 +3880,7 @@ func (s Echo_echo_Params) HasIn() bool {
 
 func (s Echo_echo_Params) InBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(0)
-	if err != nil {
-		return nil, err
-	}
-	d := p.Data()
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytes(), err
 }
 
 func (s Echo_echo_Params) SetIn(v string) error {
@@ -3985,14 +3943,7 @@ func (s Echo_echo_Results) HasOut() bool {
 
 func (s Echo_echo_Results) OutBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(0)
-	if err != nil {
-		return nil, err
-	}
-	d := p.Data()
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytes(), err
 }
 
 func (s Echo_echo_Results) SetOut(v string) error {
@@ -4560,14 +4511,7 @@ func (s Defaults) HasText() bool {
 
 func (s Defaults) TextBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(0)
-	if err != nil {
-		return nil, err
-	}
-	d := p.DataDefault([]byte("foo" + "\x00"))
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytesDefault("foo"), err
 }
 
 func (s Defaults) SetText(v string) error {
@@ -4670,14 +4614,7 @@ func (s BenchmarkA) HasName() bool {
 
 func (s BenchmarkA) NameBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(0)
-	if err != nil {
-		return nil, err
-	}
-	d := p.Data()
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytes(), err
 }
 
 func (s BenchmarkA) SetName(v string) error {
@@ -4708,14 +4645,7 @@ func (s BenchmarkA) HasPhone() bool {
 
 func (s BenchmarkA) PhoneBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(1)
-	if err != nil {
-		return nil, err
-	}
-	d := p.Data()
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytes(), err
 }
 
 func (s BenchmarkA) SetPhone(v string) error {

internal/demo/books/books.capnp.go

@@ -36,14 +36,7 @@ func (s Book) HasTitle() bool {
 
 func (s Book) TitleBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(0)
-	if err != nil {
-		return nil, err
-	}
-	d := p.Data()
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytes(), err
 }
 
 func (s Book) SetTitle(v string) error {

list.go

@@ -330,11 +330,7 @@ func (l TextList) BytesAt(i int) ([]byte, error) {
 	if err != nil {
 		return nil, err
 	}
-	b := p.Data()
-	if len(b) == 0 {
-		return b, nil
-	}
-	return b[:len(b)-1 : len(b)], nil
+	return p.TextBytes(), nil
 }
 
 // Set sets the i'th string in the list to v.

pointer.go

@@ -105,22 +105,56 @@ func (p Ptr) Interface() Interface {
 // Text attempts to convert p into Text, returning an empty string if
 // p is not a valid 1-byte list pointer.
 func (p Ptr) Text() string {
-	return p.TextDefault("")
+	b, ok := p.text()
+	if !ok {
+		return ""
+	}
+	return string(b)
 }
 
 // TextDefault attempts to convert p into Text, returning def if p is
 // not a valid 1-byte list pointer.
 func (p Ptr) TextDefault(def string) string {
-	if !isOneByteList(p) {
+	b, ok := p.text()
+	if !ok {
 		return def
 	}
+	return string(b)
+}
+
+// TextBytes attempts to convert p into Text, returning nil if p is not
+// a valid 1-byte list pointer.  It returns a slice directly into the
+// segment.
+func (p Ptr) TextBytes() []byte {
+	b, ok := p.text()
+	if !ok {
+		return nil
+	}
+	return b
+}
+
+// TextBytesDefault attempts to convert p into Text, returning def if p
+// is not a valid 1-byte list pointer.  It returns a slice directly into
+// the segment.
+func (p Ptr) TextBytesDefault(def string) []byte {
+	b, ok := p.text()
+	if !ok {
+		return []byte(def)
+	}
+	return b
+}
+
+func (p Ptr) text() (b []byte, ok bool) {
+	if !isOneByteList(p) {
+		return nil, false
+	}
 	l := p.List()
-	b := l.seg.slice(l.off, Size(l.length))
+	b = l.seg.slice(l.off, Size(l.length))
 	if len(b) == 0 || b[len(b)-1] != 0 {
 		// Text must be null-terminated.
-		return def
+		return nil, false
 	}
-	return string(b[:len(b)-1])
+	return b[:len(b)-1 : len(b)], true
 }
 
 // Data attempts to convert p into Data, returning nil if p is not a

std/capnp/json/json.capnp.go

@@ -97,14 +97,7 @@ func (s JsonValue) HasString() bool {
 
 func (s JsonValue) StringBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(0)
-	if err != nil {
-		return nil, err
-	}
-	d := p.Data()
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytes(), err
 }
 
 func (s JsonValue) SetString(v string) error {
@@ -251,14 +244,7 @@ func (s JsonValue_Field) HasName() bool {
 
 func (s JsonValue_Field) NameBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(0)
-	if err != nil {
-		return nil, err
-	}
-	d := p.Data()
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytes(), err
 }
 
 func (s JsonValue_Field) SetName(v string) error {
@@ -350,14 +336,7 @@ func (s JsonValue_Call) HasFunction() bool {
 
 func (s JsonValue_Call) FunctionBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(0)
-	if err != nil {
-		return nil, err
-	}
-	d := p.Data()
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytes(), err
 }
 
 func (s JsonValue_Call) SetFunction(v string) error {

std/capnp/rpc/rpc.capnp.go

@@ -2260,14 +2260,7 @@ func (s Exception) HasReason() bool {
 
 func (s Exception) ReasonBytes() ([]byte, error) {
 	p, err := s.Struct.Ptr(0)
-	if err != nil {
-		return nil, err
-	}
-	d := p.Data()
-	if len(d) == 0 {
-		return d, nil
-	}
-	return d[:len(d)-1], nil
+	return p.TextBytes(), err
 }
 
 func (s Exception) SetReason(v string) error {