Skip to content

Commit

Permalink
Remove a redundant condition
Browse files Browse the repository at this point in the history
  • Loading branch information
mame committed Nov 9, 2021
1 parent 64007fc commit 3628616
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions class.c
Expand Up @@ -1349,7 +1349,7 @@ class_descendants_recursive(VALUE klass, VALUE v)
struct subclass_traverse_data *data = (struct subclass_traverse_data *) v;

if (BUILTIN_TYPE(klass) == T_CLASS && !FL_TEST(klass, FL_SINGLETON)) {
if (data->buffer && (data->count < data->maxcount || data->maxcount == -1)) {
if (data->buffer && data->count < data->maxcount) {
data->buffer[data->count] = klass;
}
data->count++;
Expand Down Expand Up @@ -1383,7 +1383,7 @@ rb_class_descendants(VALUE klass)
// estimate the count of subclasses
rb_class_foreach_subclass(klass, class_descendants_recursive, (VALUE) &data);

// this allocation may cause GC which may reduce the subclasses
// the following allocation may cause GC which may change the number of subclasses
data.buffer = ALLOC_N(VALUE, data.count);
data.maxcount = data.count;
data.count = 0;
Expand Down

0 comments on commit 3628616

Please sign in to comment.