Project

General

Profile

Actions

Bug #7667

closed

Should Refinements be enabled by default?

Added by shugo (Shugo Maeda) over 11 years ago. Updated over 11 years ago.

Status:
Closed
Target version:
-
ruby -v:
ruby 2.0.0dev (2013-01-07 trunk 38719) [i686-linux]
Backport:
[ruby-core:51286]

Description

Currently, `require "refinement"' is needed to use Refinements, but I've heard from Matz that he wants Refinements to be enabled by default.
Is it right, Matz?

If so, I'm worried that other implementors might not want to enable Refinements by default, because it's an experimental feature.

Updated by matz (Yukihiro Matsumoto) over 11 years ago

  • Assignee changed from matz (Yukihiro Matsumoto) to shugo (Shugo Maeda)

Correct. I think we don't have to worry about other implementations if it's marked as EXPERIMENTAL in the NEWS file.

Matz.

Updated by shugo (Shugo Maeda) over 11 years ago

matz (Yukihiro Matsumoto) wrote:

Correct. I think we don't have to worry about other implementations if it's marked as EXPERIMENTAL in the NEWS file.

I think a warning should be shown when Module#refine or main.using is called at the first time.

Actions #3

Updated by shugo (Shugo Maeda) over 11 years ago

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

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


  • eval.c (Init_eval): enable Refinements by default.
    [ruby-core:51286] [Bug #7667]

  • eval.c (rb_mod_refine, top_using): show a warning when
    Module#refine or main.using is called at the first time.

  • ext/refinement/*: removed the extension library "refinement".

  • test/ruby/test_refinement.rb: fix for the above changes.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0