search

lesismal / nbio

Pure Go 1000k+ connections solution, support tls/http1.x/websocket and basically compatible with net/http, with high-performance and low memory cost, non-blocking, event-driven, easy-to-use.
MIT License
2.17k stars 153 forks source link

feat: update MaxWriteBufferSize comment and MaxWriteBufferSize default check #427

Closed byene0923 closed 5 months ago

byene0923 commented 5 months ago

hello，i have read the talk in issues 264，but I found comment and check for MaxWriteBufferSize may be not right？I have done follow things。

  1. update outdated comment for MaxWriteBufferSize
  2. set DefaultMaxWriteBufferSize 0 by default
  3. add negative number check for MaxWriteBufferSize in NewEngine()