Feature #7508

Clean up some more calls to rb_intern()

Added by charliesome (Charlie Somerville) over 7 years ago. Updated over 6 years ago.

Target version:


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.


clean_up_rb_interns.patch (16.1 KB) clean_up_rb_interns.patch charliesome (Charlie Somerville), 12/04/2012 06:54 PM

Updated by usa (Usaku NAKAMURA) over 7 years ago

  • Category set to core
  • Status changed from Open to Assigned
  • Assignee set to charliesome (Charlie Somerville)
  • Target version set to 2.0.0

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


Updated by Glass_saga (Masaki Matsushita) over 7 years ago

  • Tracker changed from Bug to Feature

Updated by charliesome (Charlie Somerville) over 7 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) over 7 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 6 years ago

Why not idCmp and idPow in array.c?

Updated by charliesome (Charlie Somerville) over 6 years ago

  • Status changed from Assigned to Closed

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

Also available in: Atom PDF