[Groonga-commit] groonga/groonga at 57e6f56 [master] grn_ts: suppress warnings about implicit pointer type casting

Back to archive index

susumu.yata null+****@clear*****
Wed Oct 7 15:28:08 JST 2015


susumu.yata	2015-10-07 15:28:08 +0900 (Wed, 07 Oct 2015)

  New Revision: 57e6f56ce3f4972bf6c320baa2a319f6a316f9d9
  https://github.com/groonga/groonga/commit/57e6f56ce3f4972bf6c320baa2a319f6a316f9d9

  Message:
    grn_ts: suppress warnings about implicit pointer type casting

  Modified files:
    lib/ts.c

  Modified: lib/ts.c (+14 -19)
===================================================================
--- lib/ts.c    2015-10-07 15:14:11 +0900 (15598d1)
+++ lib/ts.c    2015-10-07 15:28:08 +0900 (2c6cb64)
@@ -4849,7 +4849,7 @@ grn_ts_expr_parser_tokenize_start(grn_ctx *ctx, grn_ts_expr_parser *parser,
   if (rc != GRN_SUCCESS) {
     return rc;
   }
-  *token = new_token;
+  *token = (grn_ts_expr_token *)new_token;
   return GRN_SUCCESS;
 }
 
@@ -4863,7 +4863,7 @@ grn_ts_expr_parser_tokenize_end(grn_ctx *ctx, grn_ts_expr_parser *parser,
   if (rc != GRN_SUCCESS) {
     return rc;
   }
-  *token = new_token;
+  *token = (grn_ts_expr_token *)new_token;
   return GRN_SUCCESS;
 }
 
@@ -5011,7 +5011,7 @@ grn_ts_expr_parser_tokenize_bridge(grn_ctx *ctx, grn_ts_expr_parser *parser,
   if (rc != GRN_SUCCESS) {
     return rc;
   }
-  *token = new_token;
+  *token = (grn_ts_expr_token *)new_token;
   return GRN_SUCCESS;
 }
 
@@ -5026,7 +5026,7 @@ grn_ts_expr_parser_tokenize_bracket(grn_ctx *ctx, grn_ts_expr_parser *parser,
   if (rc != GRN_SUCCESS) {
     return rc;
   }
-  *token = new_token;
+  *token = (grn_ts_expr_token *)new_token;
   return GRN_SUCCESS;
 }
 
@@ -5105,9 +5105,7 @@ grn_ts_expr_parser_tokenize_op(grn_ctx *ctx, grn_ts_expr_parser *parser,
   grn_ts_expr_op_token *new_token;
   switch (str.ptr[0]) {
     case '+': case '-': {
-      // FIXME: Suppress a warning.
-      rc = grn_ts_expr_parser_tokenize_sign(ctx, parser, str, &new_token);
-      break;
+      return grn_ts_expr_parser_tokenize_sign(ctx, parser, str, token);
     }
 #define GRN_TS_EXPR_PARSER_TOKENIZE_OP_CASE(label, TYPE, EQUAL_TYPE)\
   case label: {\
@@ -5141,15 +5139,14 @@ grn_ts_expr_parser_tokenize_op(grn_ctx *ctx, grn_ts_expr_parser *parser,
     GRN_TS_EXPR_PARSER_TOKENIZE_OP_CASE('|', BITWISE_OR, LOGICAL_OR)
 #undef GRN_TS_EXPR_PARSER_TOKENIZE_OP_CASE
     case '=': {
-      if ((str.size >= 2) && (str.ptr[1] == '=')) {
-        token_str.size = 2;
-        rc = grn_ts_expr_op_token_open(ctx, token_str, GRN_TS_OP_EQUAL,
-                                       &new_token);
-      } else {
-        GRN_TS_ERR(GRN_INVALID_FORMAT, "single equal not available: =\"%.*s\"",
-                   (int)str.size, str.ptr);
-        rc = ctx->rc;
+      if ((str.size < 2) || (str.ptr[1] != '=')) {
+        GRN_TS_ERR_RETURN(GRN_INVALID_FORMAT,
+                          "single equal not available: =\"%.*s\"",
+                          (int)str.size, str.ptr);
       }
+      token_str.size = 2;
+      rc = grn_ts_expr_op_token_open(ctx, token_str, GRN_TS_OP_EQUAL,
+                                     &new_token);
       break;
     }
 #define GRN_TS_EXPR_PARSER_TOKENIZE_OP_CASE(label, TYPE)\
@@ -5166,10 +5163,8 @@ grn_ts_expr_parser_tokenize_op(grn_ctx *ctx, grn_ts_expr_parser *parser,
     GRN_TS_EXPR_PARSER_TOKENIZE_OP_CASE('%', MODULUS)
 #undef GRN_TS_EXPR_PARSER_TOKENIZE_OP_CASE
     default: {
-      GRN_TS_ERR(GRN_INVALID_FORMAT, "invalid character: \"%.*s\"",
-                 (int)str.size, str.ptr);
-      rc = ctx->rc;
-      break;
+      GRN_TS_ERR_RETURN(GRN_INVALID_FORMAT, "invalid character: \"%.*s\"",
+                        (int)str.size, str.ptr);
     }
   }
   if (rc != GRN_SUCCESS) {
-------------- next part --------------
HTML����������������������������...
下载 



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