ActiveSupport: alternative Block execution on empty Enumerable

Attached a patch for a more efficient implementation: http://gist.github.com/163274

@Luca: In the docs you miss the 'do' keyword: “end.else” => “end.else do”. Besides that; verified and I like it! +1

Thanks Eloy, I just fixed the rdoc issue and added a couple of test cases.

I could definitely see myself using something like this. I would definitely change the name from else though. It's bad practice to use keywords as method names and else provides no context when used by itself. +1 (with some changes)

@blatyo which name do you suggest?

i do not think the use of else is bad within this context. it reads nice:

with.each do this end.else do that end

using it "standalone" does not make much sense i.m.O.

elements.else do
... end

an

if elememts.blank?
... end

is and will be better - regardless of the naming of the method...

My take on it is: when_blank