[Groonga-commit] nroonga/nroonga at 9dba25b [master] Fix using Nan::CopyBuffer

Back to archive index

abetomo null+****@clear*****
Fri Sep 8 17:56:15 JST 2017


abetomo	2017-09-08 17:56:15 +0900 (Fri, 08 Sep 2017)

  New Revision: 9dba25b391d88a8844ddbb205981fa5925032dca
  https://github.com/nroonga/nroonga/commit/9dba25b391d88a8844ddbb205981fa5925032dca

  Merged e785504: Merge pull request #19 from abetomo/Fix_using_nan_copy_buffer

  Message:
    Fix using Nan::CopyBuffer
    
    reference:
    https://github.com/nroonga/nroonga/commit/c4e8c9ae2b90b9ce1457ebfde1b5336993027c2d

  Modified files:
    src/nroonga.cc

  Modified: src/nroonga.cc (+2 -5)
===================================================================
--- src/nroonga.cc    2017-09-08 15:18:16 +0900 (bf0cd21)
+++ src/nroonga.cc    2017-09-08 17:56:15 +0900 (6c8bd24)
@@ -116,12 +116,9 @@ void Database::CommandAfter(uv_work_t* req) {
         Nan::New(baton->context.errbuf).ToLocalChecked());
     argv[1] = Nan::Null();
   } else {
-    grn_obj body;
-    GRN_TEXT_INIT(&body, GRN_OBJ_DO_SHALLOW_COPY);
-    GRN_TEXT_SET(&baton->context, &body, baton->result, baton->result_length);
     argv[0] = Nan::Null();
-    argv[1] = Nan::New(GRN_TEXT_VALUE(&body)).ToLocalChecked();
-    GRN_OBJ_FIN(&baton->context, &body);
+    argv[1] = Nan::CopyBuffer(baton->result, baton->result_length)
+        .ToLocalChecked();
   }
   Nan::MakeCallback(Nan::GetCurrentContext()->Global(),
                     Nan::New<v8::Function>(baton->callback),
-------------- next part --------------
HTML����������������������������...
URL: https://lists.osdn.me/mailman/archives/groonga-commit/attachments/20170908/5fc68035/attachment-0001.htm 



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