Project

General

Profile

Misc #10249

[PATCH] set rb_event_flag_t to uint32_t

Added by normalperson (Eric Wong) over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
[ruby-core:65083]

Description

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...


Files

Updated by ko1 (Koichi Sasada) over 5 years ago

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

Updated by normalperson (Eric Wong) over 5 years ago

ko1@atdot.net 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 :<

Updated by ko1 (Koichi Sasada) over 5 years ago

  • Status changed from Open to Closed

Also available in: Atom PDF