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
Server should not clear writeBuffer in every flush #91
Conversation
I used SourceTree for rebasing with master but Github Mac client for commit. So in this pull request I see last commit changes also, I think this is related to using different clients for committing and rebasing. |
Please rebase against master |
Done. |
this.writeBufferLengths.push(writeBufferObject.length); | ||
this.emit('flush', writeBufferObject); | ||
this.server.emit('flush', this, writeBufferObject); | ||
this.transport.send(writeBufferObject); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's really hard for me to follow the logic here :\ Variable names are too long as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh! Ok, So please let me to change the variable names.
Ok. Please check the code again. |
var msg = "boo"; | ||
|
||
socket.on('drain', function() { | ||
expect(socket.writeBuffer.length).not.to.be(0); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
double indent
Ops, Excuse me for that. Fixed. |
Slow upgrade should not take down server.
add new test for clean writeBuffer in nextTick.
New commits added for issue #89. |
fix "TypeError: Object #<Object> has no method 'global'"
Issue #59