qbit/go
1
0
Fork 0
mirror of https://github.com/golang/go synced 2024-10-01 11:28:34 -06:00
Code Releases Activity

io: document that Readers and Writers must not retain buffers

Browse Source 
There are both many callers and many implementations of these
interfaces, so make the contract explicit. Callers generally
assume this, and at least the standard library and other
implementations obey this, but it's never stated explicitly,
making it somewhat risky to assume.

LGTM=gri, rsc
R=golang-codereviews, gri
CC=golang-codereviews, r, rsc
https://golang.org/cl/132150043
This commit is contained in:
Brad Fitzpatrick 2014-08-25 21:38:39 -07:00
parent 3c466dfea4
commit 87b452537a
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/pkg/io/io.go
View File

@ -63,7 +63,7 @@ var ErrNoProgress = errors.New("multiple Read calls return no data or error")
//
// Implementations of Read are discouraged from returning a
// zero byte count with a nil error, and callers should treat
// that situation as a no-op.
// that situation as a no-op. Implementations must not retain p.
type Reader interface {
Read(p []byte) (n int, err error)
}
@ -75,6 +75,8 @@ type Reader interface {
// and any error encountered that caused the write to stop early.
// Write must return a non-nil error if it returns n < len(p).
// Write must not modify the slice data, even temporarily.
//
// Implementations must not retain p.
type Writer interface {
Write(p []byte) (n int, err error)
}
@ -192,6 +194,8 @@ type WriterTo interface {
//
// Clients of ReadAt can execute parallel ReadAt calls on the
// same input source.
//
// Implementations must not retain p.
type ReaderAt interface {
ReadAt(p []byte, off int64) (n int, err error)
}
@ -209,6 +213,8 @@ type ReaderAt interface {
//
// Clients of WriteAt can execute parallel WriteAt calls on the same
// destination if the ranges do not overlap.
//
// Implementations must not retain p.
type WriterAt interface {
WriteAt(p []byte, off int64) (n int, err error)
}