Skip to content

add readCall/writeReply in V1.0 #58

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

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