diff options
| author | Jean Boussier <[email protected]> | 2025-05-27 11:51:29 +0200 |
|---|---|---|
| committer | Jean Boussier <[email protected]> | 2025-05-27 12:45:24 +0200 |
| commit | e535f8248b1ad9f18cfc8134dd7e8056d97a6244 () | |
| tree | 98be43b7ffc1bea7eef31c0e05136a9e3e20884a /ext | |
| parent | cc48fcdff22689cdca04181105582150b01904a1 (diff) | |
Get rid of `rb_shape_id(rb_shape_t *)`
We should avoid conversions from `rb_shape_t *` into `shape_id_t` outside of `shape.c` as the short term goal is to have `shape_id_t` contain tags.
Notes: Merged: https://.com/ruby/ruby/pull/13448
| -rw-r--r-- | ext/objspace/objspace_dump.c | 8 |
1 files changed, 4 insertions, 4 deletions
@@ -784,15 +784,15 @@ objspace_dump(VALUE os, VALUE obj, VALUE output) } static void -shape_i(rb_shape_t *shape, void *data) { struct dump_config *dc = (struct dump_config *)data; - shape_id_t shape_id = rb_shape_id(shape); if (shape_id < dc->shapes_since) { return; } dump_append(dc, "{\"address\":"); dump_append_ref(dc, (VALUE)shape); @@ -855,7 +855,7 @@ objspace_dump_all(VALUE os, VALUE output, VALUE full, VALUE since, VALUE shapes) } if (RTEST(shapes)) { - rb_shape_each_shape(shape_i, &dc); } /* dump all objects */ @@ -872,7 +872,7 @@ objspace_dump_shapes(VALUE os, VALUE output, VALUE shapes) dump_output(&dc, output, Qfalse, Qnil, shapes); if (RTEST(shapes)) { - rb_shape_each_shape(shape_i, &dc); } return dump_result(&dc); } |