Bug #3723

Remove unused control frame variable: rb_control_frame_t.block_iseq

Added by Peter Weldon over 5 years ago. Updated over 4 years ago.

Assignee:Koichi Sasada
ruby -v:ruby 1.9.3dev (2010-08-19 trunk 29056) [i386-mswin32_100] Backport:


rb_control_frame_t.block_iseq does not appear to serve any purpose and can possibly be removed.


#1 Updated by Yui NARUSE over 5 years ago

  • Status changed from Open to Assigned
  • Assignee set to Koichi Sasada

As you said, it seems useless variable.

But if block_iseq of rb_control_frame_t is simply removed, it breaks ABI compatibility.

I assign this to ko1.

#2 Updated by Yui NARUSE over 5 years ago

  • Status changed from Assigned to Rejected

I talked this with ko1.
It is used from rb_block_t.iseq, so it can't be removed.

#3 Updated by Peter Weldon over 5 years ago

I apologise, I did not understand the connection between rb_control_frame_t and rb_block_t. I see now that these macros are used to access rb_control_frame_t members via a rb_block_t:

#define RUBY_VM_GET_BLOCK_PTR_IN_CFP(cfp) ((rb_block_t *)(&(cfp)->self))
((rb_control_frame_t *)((VALUE *)(b) - 5))

The rb_control_frame_t comments in vm_core.h now also make sense to me. Thank you for helping me understand.


Also available in: Atom PDF