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

Replace assert with RUBY_ASSERT in thread_sync.c

Browse Source 
assert does not print the bug report, only the file and line number of
the assertion that failed. RUBY_ASSERT prints the full bug report, which
makes it much easier to debug.
This commit is contained in:
Peter Zhu 2024-02-08 09:45:28 -05:00
parent 9a2b692249
commit 80700f453e
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
thread_sync.c
View File

@ -854,7 +854,7 @@ raise_closed_queue_error(VALUE self)
static VALUE static VALUE
queue_closed_result(VALUE self, struct rb_queue *q) queue_closed_result(VALUE self, struct rb_queue *q)
{ {
assert(queue_length(self, q) == 0); RUBY_ASSERT(queue_length(self, q) == 0);
return Qnil; return Qnil;
} }
@ -1081,8 +1081,8 @@ queue_do_pop(VALUE self, struct rb_queue *q, int should_block, VALUE timeout)
else { else {
rb_execution_context_t *ec = GET_EC(); rb_execution_context_t *ec = GET_EC();
assert(RARRAY_LEN(q->que) == 0); RUBY_ASSERT(RARRAY_LEN(q->que) == 0);
assert(queue_closed_p(self) == 0); RUBY_ASSERT(queue_closed_p(self) == 0);
struct queue_waiter queue_waiter = { struct queue_waiter queue_waiter = {
.w = {.self = self, .th = ec->thread_ptr, .fiber = nonblocking_fiber(ec->fiber_ptr)}, .w = {.self = self, .th = ec->thread_ptr, .fiber = nonblocking_fiber(ec->fiber_ptr)},