[ruby-core:64875] [ruby-trunk - misc #10207] [Closed] [PATCH cleanup] compile.c: remove needless ID2SYM/SYM2ID conversions

Issue #10207 has been updated by Koichi Sasada.

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.


----------------------------------------
misc #10207: [PATCH cleanup] compile.c: remove needless ID2SYM/SYM2ID conversions
https://bugs.ruby-lang.org/issues/10207#change-48738

* Author: Eric Wong
* Status: Closed
* Priority: Normal
* Assignee: Eric Wong
* Category: core
* Target version: current: 2.2.0
----------------------------------------
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)


-- 
https://bugs.ruby-lang.org/