[Groonga-commit] nroonga/nroonga at 77c5262 [master] Fix using Nan::New instead of Nan::CopyBuffer

Back to archive index

abetomo null+****@clear*****
Wed Sep 20 15:42:07 JST 2017


abetomo	2017-09-20 15:42:07 +0900 (Wed, 20 Sep 2017)

  New Revision: 77c5262be62e99f9ed71671afd7cfd6a8a48ff1d
  https://github.com/nroonga/nroonga/commit/77c5262be62e99f9ed71671afd7cfd6a8a48ff1d

  Merged 64cbbc7: Merge pull request #26 from abetomo/fix_using_nan_new_instead_of_nan_copy_buffer

  Message:
    Fix using Nan::New instead of Nan::CopyBuffer

  Modified files:
    src/nroonga.cc

  Modified: src/nroonga.cc (+2 -4)
===================================================================
--- src/nroonga.cc    2017-09-20 10:24:05 +0900 (a1c7e66)
+++ src/nroonga.cc    2017-09-20 15:42:07 +0900 (1575a65)
@@ -117,8 +117,7 @@ void Database::CommandAfter(uv_work_t* req) {
     argv[1] = Nan::Null();
   } else {
     argv[0] = Nan::Null();
-    argv[1] = Nan::CopyBuffer(baton->result, baton->result_length)
-        .ToLocalChecked();
+    argv[1] = Nan::New(baton->result, baton->result_length).ToLocalChecked();
   }
   Nan::MakeCallback(Nan::GetCurrentContext()->Global(),
                     Nan::New<v8::Function>(baton->callback),
@@ -199,8 +198,7 @@ void Database::CommandSyncString(
     return;
   }
 
-  info.GetReturnValue().Set(
-      Nan::CopyBuffer(result, result_length).ToLocalChecked()->ToString());
+  info.GetReturnValue().Set(Nan::New(result, result_length).ToLocalChecked());
 }
 
 void InitNroonga(v8::Local<v8::Object> exports) {
-------------- next part --------------
HTML����������������������������...
URL: https://lists.osdn.me/mailman/archives/groonga-commit/attachments/20170920/6b1b36ce/attachment-0001.htm 



More information about the Groonga-commit mailing list
Back to archive index