Fix the warnings when compiling harfbuzz with clang in Android

Fix the warnings when compiling harfbuzz with clang in Android, the warnings were:

third_party/harfbuzz/contrib/harfbuzz-freetype.c:26:57: error: incompatible
      pointer types passing 'size_t *' (aka 'unsigned int *') to parameter of
      type 'ssize_t *' (aka 'long *') [-Werror,-Wincompatible-pointer-types]
    const uint32_t cp = utf16_to_code_point(chars, len, &i);
                                                        ^~
third_party/harfbuzz/contrib/harfbuzz-unicode.h:22:74: note: passing argument to
      parameter 'iter' here
uint32_t utf16_to_code_point(const uint16_t *chars, size_t len, ssize_t *iter);
                                                                         ^
third_party/harfbuzz/contrib/harfbuzz-freetype.c:58:57: error: incompatible
      pointer types passing 'size_t *' (aka 'unsigned int *') to parameter of
      type 'ssize_t *' (aka 'long *') [-Werror,-Wincompatible-pointer-types]
    const uint32_t cp = utf16_to_code_point(chars, len, &i);
                                                        ^~
third_party/harfbuzz/contrib/harfbuzz-unicode.h:22:74: note: passing argument to
      parameter 'iter' here
uint32_t utf16_to_code_point(const uint16_t *chars, size_t len, ssize_t *iter);
                                 

BUG=143931
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=156760

[michaelbai, 2012-09-13 17:52:44]

On 2012/09/12 17:55:53, michaelbai wrote:

Ping

[agl, 2012-09-13 17:59:03]

lgtm

https://chromiumcodereview.appspot.com/10928152/diff/1/third_party/harfbuzz/c...
File third_party/harfbuzz/contrib/harfbuzz-freetype.c (right):

https://chromiumcodereview.appspot.com/10928152/diff/1/third_party/harfbuzz/c...
third_party/harfbuzz/contrib/harfbuzz-freetype.c:26: while (i < len) {
I fear that this will generate warnings about signed/unsigned comparisons.

[michaelbai, 2012-09-13 18:23:54]

https://chromiumcodereview.appspot.com/10928152/diff/1/third_party/harfbuzz/c...
File third_party/harfbuzz/contrib/harfbuzz-freetype.c (right):

https://chromiumcodereview.appspot.com/10928152/diff/1/third_party/harfbuzz/c...
third_party/harfbuzz/contrib/harfbuzz-freetype.c:26: while (i < len) {
I didn't see the warning when compiling with clang, Let me see how it is going
in cq.

On 2012/09/13 17:59:03, agl wrote:
> I fear that this will generate warnings about signed/unsigned comparisons.

[commit-bot: I haz the power, 2012-09-13 18:24:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/michaelbai@chromium.org/10928152/1

[commit-bot: I haz the power, 2012-09-14 04:37:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/michaelbai@chromium.org/10928152/7001

[commit-bot: I haz the power, 2012-09-14 06:58:27]

Change committed as 156760