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
