Project

General

Profile

Actions

Feature #10755

open

Use rb_define_alias instead of rb_define_method for rb_cHash

Added by gogotanaka (Kazuki Tanaka) about 9 years ago. Updated over 2 years ago.

Status:
Open
Assignee:
-
Target version:
-
[ruby-core:67668]

Description

Hi, there.

The main reason for using rb_define_alias is generating more prefer Document and making code more readable.

Is there any reason rb_define_method is prefered to rb_define_alias?
If so, please let me know :)
If not so, I'm gonna do same thing for other part of code.

I've sorted some methods in terms of rb_define_alias being very after original.


Files

using_rb_define_method_for_rbHash.patch (3.19 KB) using_rb_define_method_for_rbHash.patch gogotanaka (Kazuki Tanaka), 01/18/2015 03:09 AM

Updated by gogotanaka (Kazuki Tanaka) about 9 years ago

Hi!

As https://github.com/ruby/ruby/pull/756/, it can make users a little bit confused.
Even some of them can think it's a mistake of documentation.

so I wanna commit this patch(using_rb_define_method_for_rbHash.patch), is there any problem?

Actions #2

Updated by hsbt (Hiroshi SHIBATA) over 2 years ago

  • Tracker changed from Misc to Feature
Actions

Also available in: Atom PDF

Like0
Like0Like0