Project

General

Profile

Actions

Bug #7495

closed

Clean up a few calls to rb_intern()

Added by Anonymous over 11 years ago. Updated over 11 years ago.

Status:
Closed
Assignee:
-
Target version:
-
ruby -v:
ruby 2.0.0dev (2012-12-02 trunk 38137) [x86_64-darwin11.4.0]
Backport:
[ruby-core:50472]

Description

This isn't really a bug, but it's not really a feature either so I wasn't sure what tracker to file this under. Apologies if I chose the wrong one.

This patch just cleans up a few calls to rb_intern() and replaces them with references to values from id.h


Files

clean_up_rb_interns.patch (6.07 KB) clean_up_rb_interns.patch Anonymous, 12/02/2012 03:00 PM
Actions #1

Updated by ko1 (Koichi Sasada) over 11 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r38143.
Charlie, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • array.c, enum.c, insns.def, io.c, numeric.c, parse.y, process.c,
    range.c: use prepared IDs.
    A patch from charliesome (Charlie Somerville).
    [Bug #7495]
  • common.mk: add dependency to id.h.
  • common.mk: replace ID_H_INCLUDES with id.h.

Updated by ko1 (Koichi Sasada) over 11 years ago

I applied your patch. Thanks!

Actions

Also available in: Atom PDF

Like0
Like0Like0