From 9b3fcfbbb9031036a9f7ba8ae1934f0805ea4d85 Mon Sep 17 00:00:00 2001 From: "S.H" Date: Sat, 7 Aug 2021 12:50:55 +0900 Subject: [PATCH] Suppress unused-result warnings * Hide read function warning in string_spec_RSTRING_PTR_read function * The type of `read` may be `ssize_t` Co-authored-by: Nobuyoshi Nakada --- spec/ruby/optional/capi/ext/string_spec.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/spec/ruby/optional/capi/ext/string_spec.c b/spec/ruby/optional/capi/ext/string_spec.c index b7bfcf8c88ab1c..feacbbdbb7eb55 100644 --- a/spec/ruby/optional/capi/ext/string_spec.c +++ b/spec/ruby/optional/capi/ext/string_spec.c @@ -397,12 +397,16 @@ VALUE string_spec_RSTRING_PTR_read(VALUE self, VALUE str, VALUE path) { rb_str_modify_expand(str, 30); rb_ary_push(capacities, SIZET2NUM(rb_str_capacity(str))); char *buffer = RSTRING_PTR(str); - read(fd, buffer, 30); + if (read(fd, buffer, 30) < 0) { + rb_syserr_fail(errno, "read"); + } rb_str_modify_expand(str, 53); rb_ary_push(capacities, SIZET2NUM(rb_str_capacity(str))); char *buffer2 = RSTRING_PTR(str); - read(fd, buffer2 + 30, 53 - 30); + if (read(fd, buffer2 + 30, 53 - 30) < 0) { + rb_syserr_fail(errno, "read"); + } rb_str_set_len(str, 53); close(fd);