Iterables.sizeOf(Iterable<?> iterable) runs into inf. loop for non-collection Iterables

[hoesler]

Iterator<?> iterator = iterable.iterator();
while (iterator.hasNext()) {
size++;
}

The .next() is missing.

By the way: Why did you bake your own Iterables etc. utility classes? I would suggest to use the guava lib for these tasks.

[joel-costigliola]

You are obviously right, we screwed up this one.

We don't want any third party library dependency in fest-assert-core or fest-util, to avoid Fest users to manage transitive dependencies (what if we use a version Guava not compatible with the one on your project ?).
If we pick Guava then Joda-time will come, then commons-lang, etc ...

By the way, if you are interested in Guava assertions, I'm starting a Fest module dedicated to Guava to provide for example Multimap assertions, check the progress here : https://github.com/joel-costigliola/fest-guava-assert.

[alexruiz]

Joel, can you take care of this one? :)

I could fix it on my side, but in the current state of my local repo I can't push changes yet.

[alexruiz]

I closed this issue by mistake! :/