Misc #10249: [PATCH] set rb_event_flag_t to uint32_t

• Status: Closed
• Assignee: ko1 (Koichi Sasada)

[normalperson (Eric Wong)]

This ensures a consistent implementation between 32 and 64-bit
platforms, as vm_trace.c limits events to 32-bits.

This is an ABI change late in the 2.2.0 development,
I'm OK if we wait until 2.3.0 for this...

[ko1 (Koichi Sasada)]

I'm not sure why it is valuable, but no problem to do it.

[normalperson (Eric Wong)]

[e-mail] wrote:

I'm not sure why it is valuable, but no problem to do it.

Committed r47759 - This can open up some tiny data structure packing
opportunities. I prefer small data structures in general, as Ruby keeps
getting bigger :<

[ko1 (Koichi Sasada)]

• Status changed from Open to Closed