From: "shugo (Shugo Maeda)" <redmine@...>
Date: 2012-03-25T00:23:45+09:00
Subject: [ruby-core:43586] [ruby-trunk - Bug #6159] Enumerable::Lazy#inspect


Issue #6159 has been updated by shugo (Shugo Maeda).


Eregon (Benoit Daloze) wrote:
> > I agree that It would be nice. However, unlike Enumerator, Enumerator::Lazy doesn't have enough information, so it needs to have some information only for inspect.
> > Is it so important to make inspect rich?
> 
> I'm not sure, but it would certainly be helpful when debugging.
> On the other hand, if that would impact significantly performance or code (maintainability), it's probably not worth it.
> 
> I guess I'm not used to a core Ruby class without a nice #inspect.

I guess it wouldn't impact performance or maintainability so much, and fixed Enumerable::Lazy#inspect so that the following code:

  (1..10).lazy.select(&:odd?).map(&:to_s).inspect

to return the following string:

  "#<Enumerator::Lazy: #<Enumerator::Lazy: #<Enumerator::Lazy: 1..10>:select>:map>"

----------------------------------------
Bug #6159: Enumerable::Lazy#inspect
https://bugs.ruby-lang.org/issues/6159#change-25083

Author: Eregon (Benoit Daloze)
Status: Closed
Priority: Normal
Assignee: shugo (Shugo Maeda)
Category: core
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2012-03-15 trunk 35042) [x86_64-darwin10.8.0]


Hello,

Enumerable::Lazy#inspect is undefined right now and relies on Enumerator#inspect:

    (1..5).lazy # => #<Enumerator::Lazy: #<Enumerator::Generator:0x00000101a2f9e8>:each>

I think it would be nice to be similar to (direct) Enumerator#inspect:

    (1..5).each # => #<Enumerator: 1..5:each>

So something like: #<Enumerator::Lazy: 1..5:each> or #<Enumerator::Lazy: #<Enumerator: 1..5:each>>

It would also be nice to show the chaining, like Enumerator does:

    (1..5).select.map.flat_map # => #<Enumerator: #<Enumerator: #<Enumerator: 1..5:select>:map>:flat_map> 

What do you think?


-- 
http://bugs.ruby-lang.org/