Project

General

Profile

Actions

Feature #7508

closed

Clean up some more calls to rb_intern()

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

Status:
Closed
Assignee:
-
Target version:
[ruby-core:50552]

Description

Related to #7495, I cleaned up a few more calls to rb_intern, replacing some with IDs defined in id.h, and others with IDs interned once on initialization and then cached.


Files

clean_up_rb_interns.patch (16.1 KB) clean_up_rb_interns.patch Anonymous, 12/04/2012 06:54 PM
Actions #1

Updated by usa (Usaku NAKAMURA) about 11 years ago

  • Category set to core
  • Status changed from Open to Assigned
  • Target version set to 2.0.0
  • Assignee set to Anonymous

IMO, it's OK.
Does anyone have another opinion?

Actions #2

Updated by Glass_saga (Masaki Matsushita) about 11 years ago

  • Tracker changed from Bug to Feature

Updated by Anonymous about 11 years ago

  • Target version changed from 2.0.0 to 2.6

Whoops I forgot about this ticket.

I'll commit it soon.

Updated by ko1 (Koichi Sasada) about 11 years ago

  • Target version changed from 2.6 to 2.1.0

I have an opinion.

I want to move all static IDs move to defs/id.def. I have no objection with this patch. But all of replace can be replaced after that.

What do you think about?

Updated by nobu (Nobuyoshi Nakada) over 10 years ago

Why not idCmp and idPow in array.c?

Updated by Anonymous over 10 years ago

  • Status changed from Assigned to Closed

nobu: I'll close this ticket - the patch is unlikely to still apply cleanly.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0Like0