[ruby-trunk - Bug #8378][Open] json/generator/generator.c: warning: array subscript has type 'char'

Issue #8378 has been reported by akr (Akira T.).


Bug #8378: json/generator/generator.c: warning: array subscript has type
‘char’
https://bugs.ruby-lang.org/issues/8378

Author: akr (Akira T.)
Status: Open
Priority: Normal
Assignee:
Category:
Target version:
ruby -v: r40599 ruby 2.1.0dev (2013-05-07) [i386-cygwin]
Backport: 1.9.3: UNKNOWN, 2.0.0: UNKNOWN

ふと、cygwin を target とする cross-compile をしてみたところ、
以下の警告を見つけました。
(たぶん cross でなくても警告されると思います)

make[2]: Entering directory
`/extdisk/chkbuild/chkbuild/tmp/build/20130507T115727Z/ruby/ext/json/generator’
compiling generator.c
generator.c: In function ‘isArrayOrObject’:
generator.c:897:5: warning: array subscript has type ‘char’
[-Wchar-subscripts]
generator.c:898:5: warning: array subscript has type ‘char’
[-Wchar-subscripts]
linking shared-object json/ext/generator.so

http://www.rubyist.net/~akr/chkbuild/debian/crossruby-trunk-cygwin/log/20130507T115727Z.log.html.gz

コードを見てみると以下のようになっていて、
char * を dereference した結果を isspace に渡しています。

895 char *p = RSTRING_PTR(string), *q = p + string_len - 1;
896 if (string_len < 2) return 0;
897 for (; p < q && isspace(*p); p++);
898 for (; q > p && isspace(*q); q–);
899 return (*p == ‘[’ && *q == ‘]’) || (*p == ‘{’ && *q == ‘}’);

char が signed な環境では、*p や *q は負になるかもしれないのでよろしくなくて、
*(unsigned char *)p とかにしたほうがいんじゃないでしょうか。