Review for CR 6728865 : Improved heuristics for Collections.disjoint() [updated]

[Ulf Zibis]

Am 23.12.2010 20:46, schrieb Mike Duigou:
> I have updated the webrev with the javadoc changes (identical to below) and Ulf's suggested code changes:
>
> http://cr.openjdk.java.net/~mduigou/6728865.3/webrev/

Thanks, I could convince you mostly.
But, IMO, the inner comments are too expatiated ("less than O(N/2)" is used 3 times).
Aren't the explanation comments from my last example clear enough and more fluently readable?
You use "ceiling(n/2)". Didn't you mean "ceiling O(N/2)" for consistence?

"//than mere Collection's,iterate on c2."
2 spaces are missing

"// One (or both) collections is/are empty. Nothing will match."
Shorter:
"// At least one collection is empty. Nothing will match."

Javadoc:
- the term '@throws NullPointerException' is duplicated, don't know if that would work ?
- there is a habit, maybe a rule, to insert 2 spaces after '.'

-Ulf