RFC: make isempty(itr) work for general iterables

[stevengj]

As mentioned in #5797, it would be nice if isempty(itr) worked for general iterable collections. The patch to implement this is trivial, and I don't see a downside to this.

[StefanKarpinski]

I like it. Can't see any drawback to this. It also probably obviates a lot of more specific definitions which are probably equivalent to this general criterion.

[StefanKarpinski]

I mean computationally equivalent.

[JeffBezanson]

In fact it is identical to the definition of isempty for String.

[toivoh]

+1

On Sun, Feb 16, 2014 at 1:02 AM, Steven G. Johnson <[email protected]

wrote:

As mentioned in #5797 #5797,
it would be nice if isempty(itr) worked for general iterable collections.

The patch to implement this is trivial, and I don't see a downside to this.

You can merge this Pull Request by running

git pull https://github.com/stevengj/julia isempty

Or view, comment on, or merge it at:

#5827
Commit Summary

• make isempty(itr) work for general iterables (as mentioned in Remove initial value for reduce() #5797)

File Changes

• M base/iterator.jlhttps://github.com/RFC: make isempty(itr) work for general iterables #5827/files#diff-0(2)

Patch Links:

• https://github.com/JuliaLang/julia/pull/5827.patch
• https://github.com/JuliaLang/julia/pull/5827.diff

Reply to this email directly or view it on GitHubhttps://github.com//pull/5827
.

[pao]

@toivoh Already merged!

[toivoh]

Ah, just saw it.

On Mon, Feb 17, 2014 at 9:25 PM, pao [email protected] wrote:

@toivoh https://github.com/toivoh Already merged!

Reply to this email directly or view it on GitHubhttps://github.com//pull/5827#issuecomment-35319621
.