golang/snappy

Incorrect comment on NewBufferedWriter?

ahimelman opened this issue · 0 comments

The comment (and docs) to NewBufferedWriter state that "Users must call Close to guarantee all data has been forwarded to the underlying io.Writer":

// The Writer returned buffers writes. Users must call Close to guarantee all

However, in looking at the Flush/Close implementations, and their usages in tests, it seems that users only need to call Flush, not Close, which would allow a caller to continue using the buffered writer.