funny / binary Goto Github PK

我觉得现在这个内存池设计得不好，以link里的codec_packet.go为例，里面的Encoder和Decoder会尽量复用[]byte，只有在序列化消息容量不够或者是接收包的时候容量不够时才会make一个新的，所以[]byte的创建频率是不高的。

现在的BufferPool比较符合的应用场景是频繁的创建销毁[]byte的场景，跟上面这种应用场景不符，如果长久持有[]byte，BufferPool里面的[]byte很快就被耗尽了并且不会被回收利用。

所以codec_packet.go里面用sync.Pool更合适，在频繁创建销毁[]byte的时候sync.Pool会尽量重用对象，在频率低的时候对象又会被释放。

对于sync.Pool不能精确分类[]byte的问题，可以用多个sync.Pool来实现。

func (writer *Writer) Flush() error {
    if flusher, ok := writer.w.(FlushWriter); writer.err != nil && ok {
        writer.err = flusher.Flush()
    }
    return writer.err
}

上面的代码是不是应该修改为

func (writer *Writer) Flush() error {
    if flusher, ok := writer.w.(FlushWriter); writer.err == nil && ok {
        writer.err = flusher.Flush()
    }
    return writer.err
}

我是新手，如果提错了，请谅解