[Groonga-commit] groonga/groonga at 75dd3c7 [master] egn: use grn_text_itoa() instead of sprintf().

Back to archive index

susumu.yata null+****@clear*****
Fri Jun 19 09:58:01 JST 2015


susumu.yata	2015-06-19 09:58:01 +0900 (Fri, 19 Jun 2015)

  New Revision: 75dd3c78178bce0f59749c7e711a8bceee511c4c
  https://github.com/groonga/groonga/commit/75dd3c78178bce0f59749c7e711a8bceee511c4c

  Message:
    egn: use grn_text_itoa() instead of sprintf().
    
    GitHub: #350

  Modified files:
    lib/egn.cpp

  Modified: lib/egn.cpp (+5 -4)
===================================================================
--- lib/egn.cpp    2015-06-19 09:54:17 +0900 (08c7c18)
+++ lib/egn.cpp    2015-06-19 09:58:01 +0900 (7f8b1e2)
@@ -3106,12 +3106,13 @@ grn_egn_select_output(grn_ctx *ctx, grn_obj *table,
               break;
             }
             case GRN_DB_WGS84_GEO_POINT: {
-              char buf[32];
               grn_egn_geo_point geo_point =
                 ((grn_egn_geo_point *)&bufs[j][0])[i];
-              int len = std::sprintf(buf, "\"%dx%d\"",
-                                     geo_point.latitude, geo_point.longitude);
-              GRN_TEXT_PUT(ctx, ctx->impl->outbuf, buf, len);
+              GRN_TEXT_PUTC(ctx, ctx->impl->outbuf, '"');
+              grn_text_itoa(ctx, ctx->impl->outbuf, geo_point.latitude);
+              GRN_TEXT_PUTC(ctx, ctx->impl->outbuf, 'x');
+              grn_text_itoa(ctx, ctx->impl->outbuf, geo_point.longitude);
+              GRN_TEXT_PUTC(ctx, ctx->impl->outbuf, '"');
               break;
             }
             default: {
-------------- next part --------------
HTML����������������������������...
Download 



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