1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Prefer to use RTEST when checking result of rb_io_wait. (#5341)

Browse Source 
* Prefer to use RTEST when checking result of `rb_io_wait`.

* Consistently use false for signifying no events ready.
This commit is contained in:
Samuel Williams 2021-12-25 02:26:06 +13:00 committed by GitHub
parent 4f74153846
commit 548c8f5f7a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
Notes: git 2025-06-06 00:31:58 +00:00 
Merged-By: ioquatix <samuel@codeotaku.com>
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ext/io/console/console.c
View File

@ -556,7 +556,7 @@ console_getch(int argc, VALUE *argv, VALUE io)
if (!(w & RB_WAITFD_IN)) return Qnil; if (!(w & RB_WAITFD_IN)) return Qnil;
# else # else
VALUE result = rb_io_wait(io, RUBY_IO_READABLE, timeout); VALUE result = rb_io_wait(io, RUBY_IO_READABLE, timeout);
if (result == Qfalse) return Qnil; if (!RTEST(result)) return Qnil;
# endif # endif
} }
else if (optp->vtime) { else if (optp->vtime) {

2
io.c
View File

@ -1483,7 +1483,7 @@ rb_io_maybe_wait(int error, VALUE io, VALUE events, VALUE timeout)
default: default:
// Non-specific error, no event is ready: // Non-specific error, no event is ready:
return RB_INT2NUM(0); return Qfalse;
} }
} }