Skip to content

Conversation

tangzhi
Copy link

@tangzhi tangzhi commented Jan 4, 2016

在v1协议下增加了 readCall/writeReply等相关方法及测试代码。

@codecov-io
Copy link

Current coverage is 92.50%

Merging #58 into master will decrease coverage by -3.29% as of e442d5a

@@            master    #58   diff @@
=====================================
  Files            7      7       
  Stmts         1022   1028     +6
  Branches         0    171   +171
  Methods          0      0       
=====================================
- Hit            979    951    -28
- Partial          0     34    +34
  Missed          43     43       

Review entire Coverage Diff as of e442d5a

Powered by Codecov. Updated on successful CI builds.

@fengmk2
Copy link
Member

fengmk2 commented Jan 5, 2016

协议文档能发一下吗?

@tangzhi
Copy link
Author

tangzhi commented Jan 6, 2016

@fengmk2
Copy link
Member

fengmk2 commented Jan 6, 2016

好的,竟然之前没看到有这个功能。

label = this.byteBuffer.getChar(this.byteBuffer.position());
}
// skip 'z' char
this.byteBuffer.position(this.byteBuffer.position() + 1);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this.byteBuffer.skip(1) 即可

@fengmk2 fengmk2 self-assigned this Jan 6, 2016
var result = {header: header, arguments: args};

while (this.byteBuffer.getChar(this.byteBuffer.position()) === 'H') {
this.byteBuffer.position(this.byteBuffer.position() + 1); //skip 'H' char
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this.byteBuffer.skip(1)

@fengmk2
Copy link
Member

fengmk2 commented Mar 10, 2016

@tangzhi can you rebase from master and run ci test again?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants