Misc #16346: Confusing macro name: RUBY_MARK_NO_PIN_UNLESS_NULL

• Status: Open
• Assignee: -

[y_kojima (Yohei Kojima)]

function rb_gc_mark_no_pin is renamed to rb_gc_mark_movable in the commit aac4d9d6c7e6b6b0742f3941b574f6006ccb5672.
But macro RUBY_MARK_NO_PIN_UNLESS_NULL is not renamed then.

Now, RUBY_MARK_NO_PIN_UNLESS_NULL calls rb_gc_mark_movable; this is a bit confusing.
So I suggest renaming RUBY_MARK_NO_PIN_UNLESS_NULL to RUBY_MARK_MOVABLE_UNLESS_NULL like this patch.