[Groonga-commit] groonga/groonga [master] [msgpack] body may exist even if not succeed

Back to archive index

null+****@clear***** null+****@clear*****
2011年 12月 20日 (火) 14:22:56 JST


Yoji SHIDARA	2011-12-20 05:22:56 +0000 (Tue, 20 Dec 2011)

  New Revision: 1d946300daa7feeeffb567b1c0538bc13edfa047

  Log:
    [msgpack] body may exist even if not succeed
    
    e.g. "check noexistent_table" returns "false" as body.
    
    refs. #1215

  Modified files:
    src/groonga.c

  Modified: src/groonga.c (+5 -1)
===================================================================
--- src/groonga.c    2011-12-20 01:21:38 +0000 (70963fd)
+++ src/groonga.c    2011-12-20 05:22:56 +0000 (def3189)
@@ -644,7 +644,11 @@ print_return_code(grn_ctx *ctx, grn_rc rc, grn_obj *head, grn_obj *body, grn_obj
       msgpack_packer_init(&header_packer, &head_writer_ctx, msgpack_buffer_writer);
 
        /* [HEAD, (BODY)] */
-      msgpack_pack_array(&header_packer, (rc == GRN_SUCCESS) ? 2 : 1);
+      if (GRN_TEXT_LEN(body) > 0) {
+        msgpack_pack_array(&header_packer, 2);
+      } else {
+        msgpack_pack_array(&header_packer, 1);
+      }
 
       /* HEAD := [rc, started, elapsed, (error, (ERROR DETAIL))] */
       header_size = 3;




Groonga-commit メーリングリストの案内
Back to archive index