Project

General

Profile

Actions

Misc #10207

closed

[PATCH cleanup] compile.c: remove needless ID2SYM/SYM2ID conversions

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


Description

Needless conversions made the code harder to follow.

Will commit in a few days unless there's objections.


Files

needless-sym-id-conv.patch (15.3 KB) needless-sym-id-conv.patch normalperson (Eric Wong), 09/06/2014 04:05 AM

Updated by ko1 (Koichi Sasada) over 9 years ago

  • Status changed from Open to Closed

Sorry for overlooking this ticket.

You are right.
Before Ruby 2.0, it should be VALUE (Symbol) because it is target of GC marking.
However, now they are not target of GC marking.

Actions

Also available in: Atom PDF

Like0
Like0