gnet-io / gnet-examples

Examples of gnet
MIT License
245 stars 77 forks source link

简化 upgrade 方法 #23

Closed dgqypl closed 7 months ago

dgqypl commented 7 months ago

在 upgrade 方法内调用 ws.Upgrade( ... 时,个人理解应该可以直接使用 buf 传入,而不用先创建一个 tmpReader,ws.Upgrade(readWrite{tmpReader ... 后再 buf.Next(skipN),不知道这样写是否有我没理解到的地方?

另一个小地方:第二个 err != nil 是否可以删除了,因为前面已经有一个 err != nil 判断了

dgqypl commented 7 months ago

简化后的 upgrade 方法,如果在数据不完整的情况下会有问题,此时 buf 中的字节应予以保留,待下次再读。