Project

General

Profile

Actions

Misc #10249

closed

[PATCH] set rb_event_flag_t to uint32_t

Added by normalperson (Eric Wong) about 8 years ago. Updated about 8 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) about 8 years ago

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

Updated by normalperson (Eric Wong) about 8 years ago

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) about 8 years ago

  • Status changed from Open to Closed
Actions

Also available in: Atom PDF

Like0
Like0Like0Like0