docker-archive / go-p9p

A modern, performant 9P library for Go.
Apache License 2.0
206 stars 50 forks source link

Fixes the MSize comparison when negociating msize with server #28

Closed simonferquel closed 8 years ago

simonferquel commented 8 years ago

The msize negociation was wrong. According to the man page http://plan9.bell-labs.com/magic/man2html/5/version, and Linux kernel 9pfs implementation http://git.kernel.org/cgit/linux/kernel/git/ericvh/v9fs.git/tree/net/9p/client.c#n980, the flow of version is:

codecov-io commented 8 years ago

Current coverage is 15.95% (diff: 0.00%)

Merging #28 into master will not change coverage

@@             master        #28   diff @@
==========================================
  Files            13         13          
  Lines          1128       1128          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits            180        180          
  Misses          900        900          
  Partials         48         48          

Powered by Codecov. Last update 5a23906...b8387ce

stevvooe commented 8 years ago

LGTM