[#24648] [Bug #1852] Enumerable's #hash Raises ArgumentError When Recursive Values are Present — Run Paint Run Run <redmine@...>

Bug #1852: Enumerable's #hash Raises ArgumentError When Recursive Values are Present

20 messages 2009/08/01
[#24649] Re: [Bug #1852] Enumerable's #hash Raises ArgumentError When Recursive Values are Present — Tanaka Akira <akr@...> 2009/08/01

In article <4a73e51b5a4f9_138119f2a982704e@redmine.ruby-lang.org>,

[#24652] Re: [Bug #1852] Enumerable's #hash Raises ArgumentError When Recursive Values are Present — Run Paint Run Run <runrun@...> 2009/08/01

> Is it valuable to implement such function?

[#24682] Re: [Bug #1852] Enumerable's #hash Raises ArgumentError When Recursive Values are Present — Tanaka Akira <akr@...> 2009/08/02

In article <67e307490908010125r6fa76654pa8e2224f714588fc@mail.gmail.com>,

[#24673] [Feature #1857] install *.h and *.inc — Roger Pack <redmine@...>

Feature #1857: install *.h and *.inc

21 messages 2009/08/01

[#24732] [Bug #1873] MatchData#[]: Omits All But Last Captures Corresponding to the Same Named Group — Run Paint Run Run <redmine@...>

Bug #1873: MatchData#[]: Omits All But Last Captures Corresponding to the Same Named Group

12 messages 2009/08/03

[#24775] [Feature #1889] Teach Onigurma Unicode 5.0 Character Properties — Run Paint Run Run <redmine@...>

Feature #1889: Teach Onigurma Unicode 5.0 Character Properties

30 messages 2009/08/05

[#24786] [Bug #1893] Recursive Enumerable#join is surprising — Jeremy Kemper <redmine@...>

Bug #1893: Recursive Enumerable#join is surprising

24 messages 2009/08/06
[#28422] [Bug #1893] Recursive Enumerable#join is surprising — Yusuke Endoh <redmine@...> 2010/03/02

Issue #1893 has been updated by Yusuke Endoh.

[#28438] Re: [Bug #1893] Recursive Enumerable#join is surprising — Yukihiro Matsumoto <matz@...> 2010/03/03

Hi,

[#24854] embedding ruby 1.9 frustration — Rolando Abarca <funkaster@...>

Hello,

12 messages 2009/08/10

[#24982] [Feature #1961] Kernel#__dir__ — Yutaka HARA <redmine@...>

Feature #1961: Kernel#__dir__

26 messages 2009/08/19
[#28898] [Feature #1961] Kernel#__dir__ — Roger Pack <redmine@...> 2010/03/23

Issue #1961 has been updated by Roger Pack.

[#28900] Re: [Feature #1961] Kernel#__dir__ — Kornelius Kalnbach <murphy@...> 2010/03/23

On 23.03.10 19:10, Roger Pack wrote:

[#25025] [Backport #1975] Backport Dir.mktmpdir — Kirk Haines <redmine@...>

Backport #1975: Backport Dir.mktmpdir

12 messages 2009/08/21

[#25041] Proposal: Simpler block format — Yehuda Katz <wycats@...>

I'd like to propose that we add the following syntax for procs in Ruby:

45 messages 2009/08/23
[#25046] Re: Proposal: Simpler block format — Caleb Clausen <caleb@...> 2009/08/23

Yehuda Katz wrote:

[#25049] Re: Proposal: Simpler block format — Yehuda Katz <wycats@...> 2009/08/23

On Sat, Aug 22, 2009 at 7:38 PM, Caleb Clausen <caleb@inforadical.net>wrote:

[#25058] Re: Proposal: Simpler block format — Yukihiro Matsumoto <matz@...> 2009/08/23

Hi,

[#25059] Re: Proposal: Simpler block format — Yehuda Katz <wycats@...> 2009/08/23

On Sun, Aug 23, 2009 at 3:33 PM, Yukihiro Matsumoto <matz@ruby-lang.org>wrote:

[#25063] Re: Proposal: Simpler block format — "David A. Black" <dblack@...> 2009/08/23

Hi --

[#25068] Re: Proposal: Simpler block format — brian ford <brixen@...> 2009/08/24

Hi,

[#25086] [Bug #1991] ruby should use twolevel namespace on OS X — Michal Suchanek <redmine@...>

Bug #1991: ruby should use twolevel namespace on OS X

12 messages 2009/08/24

[#25208] Module#prepend and Array#prepend — Yehuda Katz <wycats@...>

Matz,

23 messages 2009/08/30

[#25210] [Feature #2022] Patch for ruby-1.8.6 and openssl-1.0 — Jeroen van Meeuwen <redmine@...>

Feature #2022: Patch for ruby-1.8.6 and openssl-1.0

15 messages 2009/08/30

[#25220] [Bug #2026] String encodings are not supported by most of IO on Linux — Vit Ondruch <redmine@...>

Bug #2026: String encodings are not supported by most of IO on Linux

18 messages 2009/08/31

[ruby-core:25228] Re: Module#prepend and Array#prepend

From: Yusuke ENDOH <mame@...>
Date: 2009-08-31 18:24:30 UTC
List: ruby-core #25228
Hi,

2009/9/1 Carl Lerche <clerche@engineyard.com>:
> Yusuke,
>
> The reason that something like #prepend would be needed is so that ruby can
> provide a good way to do AOP. Your example is not exactly a 1-1 mapping onto
> the prepend feature. Instead, you would need to do this:
>
>>  module Speak
>>   def speak(words)
>>     puts words
>>   end
>>  end
>>  module Exclaim
>>   def speak(words)
>>     super("#{ words }!")
>>   end
>>  end
>>  class Person # Empty class that only includes the module
>>   include Speak
>>  end
>>  class Exclaimer < Person
>>   include Exclaim
>>  end
>
>
> If your classes are NOT empty and only include modules, then it is
> impossible to include a module "under" methods defined directly on the
> class.


Do you talk about the following behavior as a problem?

  class Person
    def speak(words)
      puts words
    end
  end
  module Exclaim
    def speak(words)
      super("#{ words }!")
    end
  end
  class Person
    include Exclaim
  end
  Person.new.speak("matz") #=> "matz", not "matz!"


> This causes pain when you are using a ruby library that does not
> assume that you will be extending classes. Rails "solved" this problem using
> alias_method_chain, but this method can cause confusion.


Thank you, now it's starting to make sense.

However, I think that module inclusion is not such a patching tool;
open class is.  In the above case, we can achieve our intent by
redefining `speak' directly:

  class Person
    def speak(words)
      puts "#{ words }!"
    end
  end
  Person.new.speak("matz") #=> "matz!"

When `speak' has a more complex definition, this way is a pain because
we cannot use super.  It is possible to solve the pain by preserving
the original definition of speak as a alias, for example, old_speak.
But I admit the solution is ugly.

Then, I guess that what we really need is a method wrapping feature:

  class Person
    refine_method(:speak) do |old_method, words|
      # old_method is a Method of original definition
      old_method.call("#{ words }!")
    end
  end
  Person.new.speak("matz") #=> "matz!"

-- 
Yusuke ENDOH <mame@tsg.ne.jp>

In This Thread