Merge pull request #4986 from dearblue/assert-io

Avoid comparing pointers and integers

Merge pull request #4986 from dearblue/assert-io
Avoid comparing pointers and integers
f32c73ad · Yukihiro "Matz" Matsumoto · GitHub · a3434399 · 49bba6c9 · f32c73ad
Unverified Commit f32c73ad authored Apr 29, 2020 by Yukihiro "Matz" Matsumoto Committed by GitHub Apr 29, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

mrbgems/mruby-io/src/io.c mrbgems/mruby-io/src/io.c +1 -1

No files found.
--- a/mrbgems/mruby-io/src/io.c
+++ b/mrbgems/mruby-io/src/io.c
@@ -1493,7 +1493,7 @@ mrb_io_readchar(mrb_state *mrb, mrb_value self)
 #endif

  mrb_get_args(mrb, "S", &buf);
-  mrb_assert(RSTRING_PTR(buf) > 0);
+  mrb_assert(RSTRING_PTR(buf) != NULL);
  mrb_str_modify(mrb, RSTRING(buf));
 #ifdef MRB_UTF8_STRING
  c = RSTRING_PTR(buf)[0];