[Groonga-commit] groonga/groonga at fec6cc6 [master] Remove wcrtomb because memset seems to be enough to initialize mbstate_t

Back to archive index

Susumu Yata null+****@clear*****
Fri Feb 9 12:49:28 JST 2018


Susumu Yata	2018-02-09 12:49:28 +0900 (Fri, 09 Feb 2018)

  New Revision: fec6cc6147610dfaa64141c17074375124655974
  https://github.com/groonga/groonga/commit/fec6cc6147610dfaa64141c17074375124655974

  Message:
    Remove wcrtomb because memset seems to be enough to initialize mbstate_t
    
    This change suppresses the following warning (gcc):
      groonga.c: In function ‘line_editor_fgets’:
      groonga.c:218:5: warning: ignoring return value of ‘wcrtomb’, declared with attribute warn_unused_result [-Wunused-result]
           wcrtomb(NULL, L'\0', &ps);
           ^

  Modified files:
    src/groonga.c

  Modified: src/groonga.c (+0 -1)
===================================================================
--- src/groonga.c    2018-02-09 12:10:30 +0900 (fb9889bd5)
+++ src/groonga.c    2018-02-09 12:49:28 +0900 (919c45b2f)
@@ -215,7 +215,6 @@ line_editor_fgets(grn_ctx *ctx, grn_obj *buf)
     mbstate_t ps;
     history_w(line_editor_history, &line_editor_history_event, H_ENTER, line);
     memset(&ps, 0, sizeof(ps));
-    wcrtomb(NULL, L'\0', &ps);
     for (i = 0; i < nchar; i++) {
       multibyte_len = wcrtomb(multibyte_buf, line[i], &ps);
       if (multibyte_len == (size_t)-1) {
-------------- next part --------------
HTML����������������������������...
URL: https://lists.osdn.me/mailman/archives/groonga-commit/attachments/20180209/59dafc02/attachment-0001.htm 



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