1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

stream_wrap: use v8::Integer::NewFromUnsigned()

Browse Source 
Use v8::Integer::NewFromUnsigned() when updating the writeQueueSize
field.

Before this commit, it used v8::Integer::New() but that takes an
int32_t. It's unlikely for a write queue to grow beyond 2**31-1 bytes
but let's use the unsigned integer constructor anyway, just in case.
This commit is contained in:
Ben Noordhuis 2013-08-10 23:25:38 +02:00
parent a20d565d9c
commit 2b5b37a3ab
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/stream_wrap.cc
View File

@ -92,8 +92,9 @@ void StreamWrap::GetFD(Local<String>, const PropertyCallbackInfo<Value>& args) {
void StreamWrap::UpdateWriteQueueSize() {
HandleScope scope(node_isolate);
object()->Set(write_queue_size_sym,
Integer::New(stream()->write_queue_size, node_isolate));
Local<Integer> write_queue_size =
Integer::NewFromUnsigned(stream()->write_queue_size, node_isolate);
object()->Set(write_queue_size_sym, write_queue_size);
}